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

0.0% odds #9

Closed
liquidnya opened this issue May 24, 2022 · 1 comment · Fixed by #39
Closed

0.0% odds #9

liquidnya opened this issue May 24, 2022 · 1 comment · Fixed by #39
Labels
bug Something isn't working

Comments

@liquidnya
Copy link

The queue can answer with liquidnya, you have a 0.0% chance of getting chosen in weighted random. which could mean that you actually have a 0% chance, since when you have 0 minutes weight time, you are not considered for getting picked.

However if you have more than 0 minutes of wait time, the queue might still say 0.0%, even though it is somewhere between 0.0% and 0.1% (maybe even between 0.0% and 0.05%).
So maybe a better phrasing for that case could be [...] you have a below 0.1% chance [...].

Also check if the chance can show as 100.0% even though someone has a slightly less than 100.0% chance.

@ToransuShoujo ToransuShoujo added the bug Something isn't working label May 24, 2022
@liquidnya
Copy link
Author

When #22 is merged, then someone can not have a 0% chance any longer, since that PR adds a wait list entry of 1 minute for people who add their level.
This does not solve this issue, but maybe it makes it easier to solve, since 0.0% now always means more than 0% (instead of differentiating between actual 0% and more than 0%).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants