Conversation
|
Once this is merged, we should probably also add it to the |
micahcarroll
left a comment
micahcarroll
left a comment
There was a problem hiding this comment.
Overall looks good to me, but we should wait on #35 to be completed before merging this!
| ## Use the human vs. human game mode. | ||
|
|
||
| With the Overcooked demo you can test the interaction between two human players. To do this, select the human keyboard input for both players and click on "Create Game", it will display "Waiting for game to start". Open another tab and navigate to http://localhost/psyturk to start the game. | ||
| The first user can be controlled from the same page, the other user from another tab (http://localhost/psiturk). | ||
|
|
There was a problem hiding this comment.
We should merge #35 before merging these changes, so we can merge master into this branch with the correctly updated human-human instructions before merging this PR.
|
Also, other thing that needs to be done before we can merge this is to have the status checks pass! I would wait for #35 to be merged (and merging |
|
Actually, we might want to deal with #34 before merging this PR, as the diffs will be much simpler before formatting is applied everywhere. |
3 participants
Added black and isort pre-commit github action