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

Fix performance calculation after contest #53

Merged
merged 8 commits into from
Apr 11, 2024
Merged

Conversation

nullchilly
Copy link
Contributor

nullchilly and others added 7 commits March 20, 2024 02:38
This reverts commit efebae1.

Nope, this is not enough because predictions are off now for the old
contest where we were fully accurate before.
We were accurate on this contest which predates fake ratings, but are
no longer after this adjustment.
This means old contests continue to have perfect prediction, and the
adjustment is applied to new contests based on the ID.
@meooow25
Copy link
Owner

Hi, I've moved the rating adjustment from the predictor to the contest cache. This way we can conditionally apply the adjustment to new contests (after fake ratings) while leaving alone the near-perfect performance values for old contests.

I will merge this after testing on CF. Unfortunately I can't do that right now since the API has been put behind the Cloudflare human check (again) (lol).

@meooow25
Copy link
Owner

Ok things seem to be alright, the result for this example is as expected.

Screenshot 2024-04-11 Standings - Hello 2024 - Codeforces

@meooow25 meooow25 merged commit 4e85ea8 into meooow25:master Apr 11, 2024
1 check passed
@meooow25
Copy link
Owner

Thanks for doing this, even if I ended up shifting the logic elsewhere :ghosthug:

@nullchilly nullchilly deleted the perf branch April 11, 2024 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unrated Users have Rating 0 for Performance Calculations
2 participants