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

Use MembershipScore.performance_total for sorting #60

Merged
merged 1 commit into from
Jun 2, 2024

Conversation

Syriiin
Copy link
Owner

@Syriiin Syriiin commented Jun 1, 2024

Why?

This should speed things up insanely.
Hopefully with this change we can get rid of the lb scores list cache.

Changes

  • Update Leaderboard.get_top_scores() and Leaderboard.get_pp_record() to use the new MembershipScore.performance_total field for sorting instead of doing potentially doing a case statement sort.

Note

Need to make sure all MembershipScore rows have populated performance_total before deploying this

@Syriiin Syriiin force-pushed the use-new-membershipscore-pp branch from b6f1c41 to 9e5799f Compare June 2, 2024 05:40
@Syriiin Syriiin force-pushed the use-new-membershipscore-pp branch from 9e5799f to 09f5851 Compare June 2, 2024 10:01
@Syriiin Syriiin merged commit 0f6cb79 into master Jun 2, 2024
3 checks passed
@Syriiin Syriiin deleted the use-new-membershipscore-pp branch June 2, 2024 14:42
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.

1 participant