Skip to content

feat: Algorithmic Tie-Breakers for Leaderboard Rankings (#192)#197

Open
nitishchaubeyy wants to merge 12 commits into
indresh404:mainfrom
nitishchaubeyy:feat/algorithmic-tie-breakers
Open

feat: Algorithmic Tie-Breakers for Leaderboard Rankings (#192)#197
nitishchaubeyy wants to merge 12 commits into
indresh404:mainfrom
nitishchaubeyy:feat/algorithmic-tie-breakers

Conversation

@nitishchaubeyy
Copy link
Copy Markdown
Contributor

Description

Resolves #192 by implementing deterministic tie-breaking logic for the RankHer leaderboard. Previously, ties in totalPoints resulted in non-deterministic ranking orders.

Key Changes:

  • Compound Sorting: Updated the Firestore query to include a hierarchical order: points.totalPoints (Primary) -> githubStats.commits (Secondary) -> githubUsername (Tertiary).
  • Deterministic Results: This ensures that every time a user refreshes the page, the rank order remains consistent across all sessions, providing a stable UX.
  • Index Requirement: Note that a new compound index is required in Firebase Firestore for this specific orderBy combination.

Fixes #192

@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ranker-hub Error Error Jun 6, 2026 10:38am

@github-actions github-actions Bot added backend Backend/Firebase related changes bug Something isn't working enhancement New feature or request frontend Frontend related changes (HTML/CSS/JS/React) gssoc GirlScript Summer of Code gssoc26 GirlScript Summer of Code 2026 nsoc NSoC NSoC'26 NSoC 2026 pending-review PR is pending review labels Jun 3, 2026
@indresh404
Copy link
Copy Markdown
Owner

merge conflict

@nitishchaubeyy
Copy link
Copy Markdown
Contributor Author

Done @indresh404

@indresh404
Copy link
Copy Markdown
Owner

Solve Merge Conflict

@indresh404
Copy link
Copy Markdown
Owner

Solve Merge Conflict

@nitishchaubeyy
Copy link
Copy Markdown
Contributor Author

Done @indresh404

@indresh404
Copy link
Copy Markdown
Owner

If your PR still shows the old failed Vercel check, please rebase (or merge the latest main branch into your branch) and push again to trigger a fresh deployment/check run.

@nitishchaubeyy
Copy link
Copy Markdown
Contributor Author

Hey @indresh404, I synced the branch with the latest main and pushed again to trigger the deployment. I also ran npm run build and npm run lint locally, and everything compiles perfectly without any errors on my end.

Since I don't have access to the Vercel logs, could you please check if it's failing due to a missing Environment Variable or a caching issue on Vercel's end?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend Backend/Firebase related changes bug Something isn't working enhancement New feature or request frontend Frontend related changes (HTML/CSS/JS/React) gssoc GirlScript Summer of Code gssoc26 GirlScript Summer of Code 2026 level:advanced level3 nsoc NSoC NSoC'26 NSoC 2026 pending-review PR is pending review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature][Level 3] Algorithmic Tie-Breakers for Leaderboard Rankings

2 participants