Skip to content

Exclude non-user-playable mods from mod filter in beatmap leaderboard #32949

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

Merged

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Apr 25, 2025

For easier understanding, substitute "non-user-playable" with "autoplay".

The reason that I'm bothering to do this is that if you put autoplay on and turn on the "Selected Mods" filter, the request will actually go through and hit web, and web will obviously return no scores. On song select that's maybe fine, even though probably unintended still, but now with the leaderboard state being global this also means gameplay gets impacted. Which also means that if you Ctrl-Enter to start a map in autoplay you're not going to get any gameplay leaderboard scores at all.

For easier understanding, substitute "non-user-playable" with
"autoplay".

The reason that I'm bothering to do this is that if you put autoplay on
and turn on the "Selected Mods" filter, the request will actually go
through and hit web, and web will obviously return no scores. On song
select that's *maybe* fine, even though probably unintended still, but
now with the leaderboard state being global this also means gameplay
gets impacted. Which also means that if you Ctrl-Enter to start a map in
autoplay you're not going to get any gameplay leaderboard scores at all.
@peppy peppy merged commit 3ee8eb2 into ppy:master Apr 28, 2025
10 checks passed
@bdach bdach deleted the exclude-non-user-playable-mods-from-leaderboard-filter branch April 28, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants