-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix mania star rating varying by order of notes in a chord #19120
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does star rating stay same when anchor is at different columns?
The test beatmap linked in the original issue and my own testing beatmaps all seem to be fixed with this change, including one where the anchor is the thing that changes between maps. |
I'm not knowledgeable enough regarding the code side of things so I will not comment on that part, but the bug being fixed is important since this was highly exploitable in terms of SR bloating. Planning to accept anyway but would love to see a detailed explanation of why the bug was happening and how this solves it in a way we're not encurring in any unwanted SR buffs/nerfs or side-effects in chord-heavy maps |
SR/PP change sheet added to OP. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm, thanks!
Resolves #4010.
This change targets notes in chords by giving them the maximum
individualStrains
column for notes in that chord. Also comes with some refactoring to clean up the code a little.Note: As I was fixing this change, an unrelated bug with the first note not counting to
individualStrain
causes a very similar looking issue on some very short maps (<10 seconds). I was planning to include it in this PR but because the effect is quite minor I thought it would be best to leave that for another day.SR/PP change spreadsheet: https://docs.google.com/spreadsheets/d/1VQlT253Cf_SCFNrBF1MI9oAlX5MA_i98HEdf4BulsZc/edit
As of b33e0f5