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.

Sign up for GitHub

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

Jump to bottom

Uncap speed OD-accuracy scaling #30088

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Uncap speed OD-accuracy scaling #30088

merged 1 commit into from
Oct 7, 2024

Conversation

stanriders
Copy link
Member

A small fix to make accuracy scaling on low ODs a bit more harsh

@stanriders stanriders requested a review from a team October 2, 2024 14:57
@peppy peppy added the next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! label Oct 4, 2024
@bdach
Copy link
Collaborator

bdach commented Oct 4, 2024

just as a sanity check I guess:

!diffcalc

@github-actions GitHub Actions
Copy link

github-actions bot commented Oct 4, 2024 •
edited
Loading

Target: #30088
Spreadsheet: https://docs.google.com/spreadsheets/d/1Dl92BTDPlazR1oqn6990PfpDuVAmo54BOSMk_0QTxVg/edit

@peppy
Copy link
Member

peppy commented Oct 7, 2024

I think the diffcalc run looks about in line with expectations?

@bdach
Copy link
Collaborator

bdach commented Oct 7, 2024

Yeah that was mostly to check that nothing egregious is happening. But it seems fine.

@bdach bdach merged commit 8496005 into ppy:master Oct 7, 2024
9 of 13 checks passed
@stanriders stanriders deleted the uncap-od-accuracy branch October 7, 2024 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@apollo-dw apollo-dw apollo-dw approved these changes

@tsunyoku tsunyoku tsunyoku approved these changes

@peppy peppy peppy approved these changes

Assignees
No one assigned
Labels
area:difficulty next release Pull requests which are almost there. We'll aim to get them in the next release, but no guarantees! size/XS
Projects
Difficulty calculation changes
Status: Deployed
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@stanriders @bdach @peppy @tsunyoku @apollo-dw