osu icon indicating copy to clipboard operation
osu copied to clipboard

Perfect doesn't work when used with Easy

Open Flan-Fan opened this issue 3 years ago • 7 comments

Type

Game behaviour

Bug description

I was playing Hazu's Hard diff of Oka Soyo - OPENING with the mods Easy, Perfect, Hidden, Flashlight and Wiggle when I noticed that the map didn't auto-retry when I missed the first note

Upon further testing, I found that the problem was when I used the mods Easy and Perfect together. I also know that this bug wasn't specific to this map, as I tried playing the Hard diff of padowo padowo by senzawa with the mods Easy and Perfect and I was able to get a score with Easy and Perfect that wasn't an SS

Screenshots or videos

Video showing bug: https://youtu.be/vOi0GZ43bY0 Screenshot of a bugged score: zJtplMA Replay: bugged score.zip

Version

2022.923.1

Logs

database.log network.log performance.log runtime.log updater.log

Flan-Fan avatar Sep 27 '22 15:09 Flan-Fan

This is because perfect mod let you fail but easy mod give you extra life

this mean sd also have this problem

cdwcgt avatar Sep 27 '22 17:09 cdwcgt

https://github.com/ppy/osu/blob/master/osu.Game/Screens/Play/Player.cs#L142

- protected virtual bool CheckModsAllowFailure() => GameplayState.Mods.OfType<IApplicableFailOverride>().All(m => m.PerformFail());
+ protected virtual bool CheckModsAllowFailure() => GameplayState.Mods.OfType<IApplicableFailOverride>().Any(m => m.PerformFail());

~~this will fix~~

need to determine whether there is IApplicableFailOverride or Any method will return false

cdwcgt avatar Sep 27 '22 17:09 cdwcgt

I'm not sure these mods should be allowed to be used together in the first place. Will wait for more opinions.

peppy avatar Sep 28 '22 08:09 peppy

You are failing if you lose all your HP. SD just makes you lose all HP once you miss, PF once you make any mistake. Then, EZ gives you N chances to fully recover your HP without failing. Personally I can't see what's wrong here - SD/PF just make HP draining harsher, they have nothing to do with its recovering.

Also, in past I played a lot with EZTP especially to have ability to make several mistakes and still pass a map. The proposed change removes this opportunity.

Feodor0090 avatar Sep 28 '22 23:09 Feodor0090

I'm not sure these mods should be allowed to be used together in the first place. Will wait for more opinions.

Personally I'm of the opinion that SD and PF should bypass the traditional failure route altogether and force failure through separate means. That way every mod (even easy/relax/autopilot) could work with SD and PF. I think if the player wants to test perfect autopilot play (so, perfect accuracy with no aim component), they should be able to do that. Same for Relax and perfect aim, there are lots of use cases for this type of implementation.

hburn7 avatar Sep 29 '22 01:09 hburn7

The key is whether these mods should be forced to fail. If easy allow to give extra life to sd and pf They will no longer mean full combo and ss This is contrary to the design of these mods. Getting them incompatible is a solution to this issues but not suitable. I think making player force fail is a good solution.

cdwcgt avatar Sep 29 '22 10:09 cdwcgt

You can just go to mod customization and set extra lives for Easy to 0, that seems to work fine. Or am I missing something?

ghost avatar Jun 02 '23 23:06 ghost

Probably closing for the same reason as https://github.com/ppy/osu/issues/29731#issuecomment-2334529002 and the comment above (setting extra lives to 0) seems like a good compromise (if extra lives <= 2 is ranked).

Joehuu avatar Sep 06 '24 17:09 Joehuu