Skip to content

Ranking improvements for developers and teams with equal impact - #103

Open
vbarhatov wants to merge 2 commits into
mainfrom
ranking-ties-share-place
Open

vbarhatov wants to merge 2 commits into
mainfrom
ranking-ties-share-place

Conversation

@vbarhatov

@vbarhatov vbarhatov commented Oct 21, 2026 •

Copy link
Copy Markdown

Developers with equal impact now share a place, and the next one skips ahead: 1, 1, 1, 4 (was 1, 2, 3, 4) instead of giving then places by First Name. Row order is unchanged.

Developers: dev table, org page, spend table, profile, CSV/Sheets export, AI summary and highlights. Shared helper in src/lib/ranking.ts.

Teams: no place numbers. When teams tie on the sort column, rows are ordered by impact (A), then by name.

No DB or scoring changes. Cached summaries and highlights keep old places until regenerated.

🤖 Generated with Claude Code

vbarhatov and others added 2 commits October 21, 2026 19:51
Equal impact now gets the same place and the next one skips ahead
(1, 1, 1, 4) instead of i + 1. Applies to the developer table, org
page, spend table, profile, CSV/Sheets export, AI summary and
highlights. Row order is unchanged.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
When teams are equal on the sort column, order them by impact (A), the
mean per active developer so team size is accounted for, then by name.
Only the row order changes; no place numbers for teams.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@vbarhatov
vbarhatov requested a review from msogin October 21, 2026 18:14
@vbarhatov vbarhatov changed the title Tied developers share a place in rankings Ranking improvements for developers and teams with the same impact Oct 21, 2026
@vbarhatov vbarhatov changed the title Ranking improvements for developers and teams with the same impact Ranking improvements for developers and teams with equal impact Oct 21, 2026
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