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.

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

Allow a few new mods and mod settings for pp #26935

Merged
merged 6 commits into from Feb 6, 2024
Merged

Conversation

bdach
Copy link
Collaborator

@bdach bdach commented Feb 1, 2024

Allow pp for "Restart on fail" setting of Sudden Death & Perfect (c114fd8)

Closes #26844.

Allow pp for Accuracy Challenge (96f66aa)

Addresses #26919.

Allow pp for "Adjust pitch" setting of Half Time / Double Time (ea76f7a)

Addresses https://discord.com/channels/188630481301012481/1097318920991559880/1202606638951825459.

Allow pp for Blinds (a84f53b)

The mod does impact pp, but it requires no extra difficulty attributes (#26935 (comment)).

Allow pp for No Scope (8df593a)

As per #26935 (comment).

@stanriders
Copy link
Member

If caching isn't a concern here, I'd propose adding some more mods:

Blinds

Long long time ago pp for blinds was implemented so it's pretty much good to go

Random

This mod doesn't require any additional work diff/ppcalc side since it only moves objects around

Mirror (osu!)

Same as random, but I'd even say it's okay to make it not affect pp at all - difference in pp and difficulty is usually negligible

Noscope

Purely visual mod, shouldn't be accounted in pp at all


If you're only "ranking" mods that don't affect difficulty at all then you can add just the last two

@bdach
Copy link
Collaborator Author

bdach commented Feb 4, 2024

Blinds, random and mirror are 100% a no-go right now. We're not calculating difficulty attributes on the fly.

@pull-request-size pull-request-size bot added size/S and removed size/L labels Feb 4, 2024
@stanriders
Copy link
Member

Blinds isn't affecting diffcalc -it's completely in pp calc

The mod does impact pp, but it requires no extra difficulty attributes
(ppy#26935 (comment)).
Deemed as not affecting difficulty or pp in
ppy#26935 (comment),
so can be treated pretty much as nomod.
@bdach
Copy link
Collaborator Author

bdach commented Feb 6, 2024

Alright I've applied for the two mods that can be applied right now.

@peppy peppy self-requested a review February 6, 2024 12:45
@peppy peppy merged commit d784934 into ppy:master Feb 6, 2024
13 of 17 checks passed
@bdach bdach deleted the new-mods-for-pp branch February 6, 2024 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

Enabling the "Restart on fail" option on the Sudden Death mod results in no pp being awarded
3 participants