-
Notifications
You must be signed in to change notification settings - Fork 206
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
Fixed Golden Cookie percentage calculation to cdf #921 #1190
Conversation
Please make the changes in the relevant files. The file you are changing should be the build output. |
Initially, I made the changes directly to the CookieMonster.js file, which I now realize wasn't the best approach. I also mistakenly pushed the entire repository, including generated files, which was not ideal. I've since reset the branch, applied the changes only to the relevant src/ files, and have pushed the updated commit. The changes should now align with the expected workflow. Please review the updated pull request. |
Sorry just saw your message. Could you also run the build? |
Do you want this to be available on the normal version? Then we also need to run |
Did my last commit run the intended function? |
Yes it did. I had a look at the code: shouldn't this be a toggle able feature? It seems like a breaking change that might not be appreciated by all users. Should we make this a setting to switch between the two calculations? |
I understand what u mean, but it would effectively switch between showing "<0%" nearly all the time and the proposed implementation (Probability that GC has spawned until now). In the end, I am indifferent to both solutions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You know what, let's just merge this!
Thanks!
Changes the Golden Cookie probability percentage to a cumulative percentage, yielding the cumulative distribution as described by @CptHampton here
resolves #921