Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Developer mode - group 7 #465

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

avipotter33
Copy link

@avipotter33 avipotter33 commented Aug 9, 2023

Submitted by Itay Hofshy, Avia Izakovitch, Gavriel Fernanadez.

We created a developer mode which contains a few features that will help the developer debug his code and work on his code. In this mode you would find:
A pause button - allows the player to stop the game, check his code (or for any other reason), and then continue the same game. In addition, it allows the developer to track the movement of his car frame by frame - stop after each movement and understand why and how it happened.
A dash board, which shows the action the server recieved, and the delay between the beginning of the driver's code and the receivement of the action. This dashboard will help the developer keep track of his code while the game is running, and also spot if a mistake happened beacuse of a problem in the code, or beacuse of a connection problem to the server.

Screenshots:
WhatsApp Image 2023-08-10 at 09 18 14
צילום מסך 2023-08-10 091416

@yaacov
Copy link
Member

yaacov commented Aug 10, 2023

@sleviim hi, please review.

@yaacov
Copy link
Member

yaacov commented Aug 10, 2023

@avipotter33 hi,

can you add screenshots of the feature ( before "Play" / after "Music" ) ?
can you update the PR description to include why it's needed to change the button from "Play" to "Music" ?

@sleviim
Copy link
Member

sleviim commented Aug 10, 2023

@yaacov @avipotter33
Thanks for the PR and the screenshots.
If it's not too much trouble, please generate a new "after" screenshot without the "Fun" button.
Other than that - looks good :)

@sleviim
Copy link
Member

sleviim commented Aug 10, 2023

I think you also uploaded the change for your feature
(The changed files not only deal with the "Play" wording as before)

@avipotter33 avipotter33 changed the title changed the Play button to Music button Developer mode - group 7 Aug 10, 2023
@itayfree
Copy link
Contributor

@sleviim hi, this is the wrong PR, the one with the "Music" is #470

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants