Skip to content

Conversation

@bdach
Copy link
Collaborator

@bdach bdach commented Oct 22, 2025

The "partial" leaderboard logic in SoloGameplayLeaderboardProvider always assumed the online fetch would request 50 scores, which is no longer the case after #33100.

Noticed in https://discord.com/channels/188630481301012481/188630652340404224/1430503761440149554.

The circumstances where this matters are pretty rare. You'd basically need to find an online leaderboard (friends / country / with selected mods) that has between 50 and 100 scores - if you do and then enter gameplay, the tracked score won't show a rank when it's last, even though it can. One such example I found that's usable right now is https://osu.ppy.sh/beatmapsets/2400092#osu/5204282, scoped to Hidden mod, which returns 63 online scores.

Because this uses the total score count as returned by web, it's probably also more accurate when a leaderboard has precisely 50 scores, but that's even more difficult to confirm, so I won't even try.

@bdach bdach self-assigned this Oct 22, 2025
@bdach bdach added area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. quick fix Tasks which were taken on because they take no time to fix labels Oct 22, 2025
@bdach bdach moved this from Next up to Pending Review in @peppy's untitled project Oct 22, 2025
…hnically could

The "partial" leaderboard logic in `SoloGameplayLeaderboardProvider`
always assumed the online fetch would request 50 scores, which is no
longer the case after ppy#33100.
@bdach bdach force-pushed the leaderboard-sometimes-incorrectly-partial branch from 040bb1b to ca8fc81 Compare October 23, 2025 06:53
@peppy peppy self-requested a review October 24, 2025 05:34
Copy link
Member

@peppy peppy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense

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

Labels

area:online functionality Deals with online fetching / sending but don't change much on a surface UI level. quick fix Tasks which were taken on because they take no time to fix size/M

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants