Improve buzzer: fair tiebreaks and quantization #62
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Someone is attempting to deploy a commit to a Personal Account owned by @cmnord on Vercel. @cmnord first needs to authorize it. |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
cmnord
reviewed
Feb 20, 2024
|
One (possibly sad) consequence of this PR - almost every prior game will become impossible to review, since the tiebreak rules will resolve ties differently, so the state will get 'stuck' at the first tie that resolves differently. The main mitigation I can think of would be versioning the game engine, add a column to 'rooms' that holds the engine version at time of creation, and then applying these new tiebreak rules only for the new engine version. It's sadly a bit complicated, though. |
neilmovva
force-pushed
the
improve-buzzer
branch
from
4824e09
to
e034fee
Compare
neilmovva
force-pushed
the
improve-buzzer
branch
3 times, most recently
from
855c34c
to
5c4c56c
Compare
Buzzer ties are now broken randomly, independently for each clue. Buzzer times are rounded to the nearest 200ms to slightly deweight buzzer racing as a game mechanic. Lockouts should still carry a penalty, otherwise buzz-spamming becomes optional.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Buzzer ties are now broken randomly, independently for each clue.
Buzzer times are rounded to the nearest 100ms to slightly deweight buzzer racing as a game mechanic.