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

Include Flashlight in total SR calculation #14753

Merged
merged 4 commits into from
Sep 21, 2021
Merged

Conversation

MBmasher
Copy link
Member

@MBmasher MBmasher commented Sep 15, 2021

Based on #13986 and #14217

#14217 needs to be merged before this.

similarly to #13986, this is just a visual change and affects only total SR.

stanriders
stanriders previously approved these changes Sep 20, 2021
Copy link
Member

@stanriders stanriders left a comment

Choose a reason for hiding this comment

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

I'm fine with this, it might be a bit controversial (like some maps being 9+ stars while being mechanically 4*) but I think it's correct way forward and we can adjust if this somehow breaks star rating medals or something else.

emu1337
emu1337 previously approved these changes Sep 21, 2021
Copy link
Member

@emu1337 emu1337 left a comment

Choose a reason for hiding this comment

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

Personally I don't think visuals belong in SR but according to the recent community survey on pp & sr, 75% of people want "subjective" skills to be included in star rating, so that would imply that this change should be accepted.

@smoogipoo smoogipoo dismissed stale reviews from emu1337 and stanriders via b9c9111 September 21, 2021 03:45
@smoogipoo smoogipoo enabled auto-merge September 21, 2021 03:46
@peppy
Copy link
Member

peppy commented Sep 21, 2021

If we're going halfway and adding FL into star rating, I'd be curious to hear thoughts on how a "complete blind" beatmap would be rated in star rating. Would it get an infinite rating? Is there an upper cap to the "memorisation" skill? Should we be calling this "Memorisation" rather than "Flashlight" to account for future similar mods (ie. if we are going to also add star rating adjustments for something like "Blinds")?

@smoogipoo smoogipoo merged commit 6ac7440 into ppy:master Sep 21, 2021
@MBmasher
Copy link
Member Author

@peppy
The Flashlight skill was written with the Flashlight mod in mind. In my opinion it's completely fine to have a skill centered around a specific mod. In fact I think it could be a good idea to even make a skill for Hidden in the future. Compare this to mods such as HR and DT, all they do is change timing between objects or certain difficulty attributes, so plays done with those mods could technically be treated as nomod plays.

Flashlight plays are not pure memory plays (as opposed to other mods such as taiko, where HDFL plays are pure memorisation), so I don't think it would be a good idea to encompass everything memory into this skill, this should stay separate.

@MBmasher MBmasher deleted the fl-total-sr branch September 21, 2021 04:34
@peppy
Copy link
Member

peppy commented Sep 21, 2021

So just for me and the 99% that hasn't attempted to understand the math here: how much would you see a 100% memorisation be worth as an addition in star rating? Is that somehow different because it's "not what flashlight does" or soemthing?

@MBmasher
Copy link
Member Author

Is that somehow different because it's "not what flashlight does" or soemthing?

Yes, flashlight is only partly memorisation. I can't provide any insight into what would happen in a 100% memorisation scenario because such scenario doesn't really exist and this Flashlight skill wasn't written to accomodate that. If one were to exist (such as blinds mod) the calculation for that needs to be separate to this.

@peppy
Copy link
Member

peppy commented Sep 21, 2021

Seems very hand-wavy to me but maybe I'm an outlier and the average player will better understand this.

@MBmasher
Copy link
Member Author

Right, I'm just trying to understand what you meant by "how much is a 100% memorisation worth in SR". Is that in the context of Flashlight?

@peppy
Copy link
Member

peppy commented Sep 21, 2021

I'm not sure. I'd just expect something to be calculated based on the size of visibility from 0 to 100% hidden. But I'm in the camp that doesn't agree this can really be represented in a good way in SR, so I'll just abstain from comment/attempt-to-understand on this occasion.

Maybe something worth factoring in when trying to explain this to the public though.

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.

5 participants