Skip to content

Award slider accuracy PP for scores set with ScoreV2 #32634

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
merged 2 commits into from
Apr 27, 2025

Conversation

Finadoggie
Copy link
Contributor

@Finadoggie Finadoggie commented Mar 30, 2025

This does not affect ranked play whatsoever. However, if sorting leaderboards by pp is implemented, this will affect local leaderboards with imported sv2 scores. Also (probably) impacts any community-made tournament pp calculators.

tl;dr this change doesn't have much reach but the performance impact is one if statement so I see no reason not to have it

@Finadoggie
Copy link
Contributor Author

meant to target pp-dev, not master

@Finadoggie Finadoggie changed the base branch from master to pp-dev March 30, 2025 06:09
@Finadoggie Finadoggie changed the title Add Support for ScoreV2 in Osu Difficulty Calculations Add Support for ScoreV2 in Standard Difficulty Calculations Mar 30, 2025
@mcendu

This comment was marked as spam.

@BunnyTub
Copy link

BunnyTub commented Apr 2, 2025

The Classic mod should still be carried in my opinion, as slider head accuracy and slider functionality differ. ScoreV2 on stable attempts to scale all hit object scoring values to a hard limit of 1 million at 1.00 Score Multiplier (excluding bonus spinner values in osu!, dendens in osu!taiko, and collected bananas in osu!catch). Taken that part from the wiki. It means that while the scoring methods are similar, they aren't the same, even with the score cap, if that makes sense.

@tsunyoku tsunyoku moved this to Pending Review in Difficulty calculation changes Apr 15, 2025
@tsunyoku tsunyoku requested a review from a team April 15, 2025 13:55
Copy link
Member

@tsunyoku tsunyoku left a comment

Choose a reason for hiding this comment

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

Sure

@tsunyoku tsunyoku changed the title Add Support for ScoreV2 in Standard Difficulty Calculations Award slider accuracy PP for scores set with ScoreV2 Apr 15, 2025
@stanriders
Copy link
Member

I'm personally not against it, but it also is pretty much just code bloat - imported sv2 scores aren't really important in any way at all, and accounting for them (especially since the difference would be like 20pp at most) sounds kinda useless

@BunnyTub
Copy link

Wouldn't call it code bloat, since it does actually matter for score and pp calculation.

@stanriders
Copy link
Member

It doesn't affect a single ranked or even lazer-first score, it'd only matter for imported from stable scorev2 scores which are super rare to begin with

@Finadoggie
Copy link
Contributor Author

it'd only matter for imported from stable scorev2 scores which are super rare to begin with

Within the vacuum of the osu client, this is true. However, the pp calculator gets used by others for stuff like tournaments for stuff like tracking tournament pp records, and has even been referenced by commentators in official tournaments.

There should be an official standard for handling scores like this properly, rather than just pretending they’re normal classic scores.

@stanriders stanriders enabled auto-merge (squash) April 27, 2025 11:37
@tsunyoku tsunyoku moved this from Pending Review to Pending Merge in Difficulty calculation changes Apr 27, 2025
@stanriders stanriders merged commit 4f29876 into ppy:pp-dev Apr 27, 2025
4 of 8 checks passed
@github-project-automation github-project-automation bot moved this from Pending Merge to Pending Deploy in Difficulty calculation changes Apr 27, 2025
@Finadoggie Finadoggie deleted the sv2 branch April 30, 2025 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Pending Deploy
Development

Successfully merging this pull request may close these issues.

5 participants