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

Prevent pause menu from interrupting network of online game #1001

Closed
MaxCWhitehead opened this issue Jun 2, 2024 · 3 comments · Fixed by #1003
Closed

Prevent pause menu from interrupting network of online game #1001

MaxCWhitehead opened this issue Jun 2, 2024 · 3 comments · Fixed by #1003
Assignees

Comments

@MaxCWhitehead
Copy link
Collaborator

MaxCWhitehead commented Jun 2, 2024

Description

Right now the pause menu will stop running game session / network session runner, this can lead to timeouts / disconnects. Perhaps we should not pause the game when pause menu is open in online play, just let it keep running.

Or could consider pausing for all players, but this is more complex + also may lead to frustration if game is repeatedly paused by other players. Rounds are fast / cheap so having other players keep going while in menu seems acceptable to me.

Somewhat related to #994 (pausing may be used to repro this, but not direct / sole cause)

@erlend-sh
Copy link
Member

Yeah, only pausing the single session seems best. We could still let the other players know that a player has paused, e.g. by showing a ⏸️ sign above their character. That way at least there’s a chance of fair play.

@MaxCWhitehead
Copy link
Collaborator Author

Yeah, only pausing the single session seems best. We could still let the other players know that a player has paused, e.g. by showing a ⏸️ sign above their character. That way at least there’s a chance of fair play.

We could also allow pause for everyone as a custom lobby setting or something down the line maybe if desired.

@MaxCWhitehead
Copy link
Collaborator Author

This is fixed, pause in online now keeps game going in background + no longer disconnects network. On full disconnects players are now also notified / given button to leave game.

github-merge-queue bot pushed a commit that referenced this issue Jun 9, 2024
… continue running (#1003)

Previously pausing in online would pause net loop and cause timeout. Now
game continues to simulate in background, and inputs are disabled while
pause menu open in online.

Dependent on fishfolk/bones#411, CI will fail
until updated.

Fixes #1001
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants