Missing hitsounds in Editor in some 300BPM beatmaps

[FukutoTojido]

Type

Game behaviour

Bug description

I noticed that hitsounds in some 300BPM beatmaps will not play normally. Some will play, some will not.

Screenshots or videos

bug2.mp4

Beatmap used in video: VINXIS - Sidetracked Day GAMMA [300]

2024-01-03.21-14-15.mp4

Beatmap used in video Noah - Deadly force - Put an end [The end.]

2024-01-03.21-17-07.mp4

Beatmap used in video: Camellia - Flamewall [ETERNAL SACRED FIRE]

Version

2023.1231.0-lazer

Logs

compressed-logs.zip

[peppy]

Age old issue of editor not running in a frame stable mode.

[peppy]

So, a few things:

Yes, this is fixed by frame stability, but only because frame stability allows replay frames to exist in a non-linear time scale. As in there are frames in the past, and frame stability will rewind to handle them correctly.

The base issue is that hitobjects are closer than 50 ms in these beatmaps, but auto generator adds a 50 ms minimum window for key-down-to-key-up:

osu/osu.Game.Rulesets.Osu/Replays/OsuAutoGenerator.cs

Line 293 in e182acf

double hEndTime = h.GetEndTime() + KEY_UP_DELAY;

I'd argue that this should be solved by fixing the auto generator to infer the next object and not hold for as long, or just reducing the key down period altogether to a value which will not cause issues like this.

Side note that DelayedMovements in autogenerator is currently never used and a dead code path...

This is a duplicate of #22353.

[peppy]

As this is a duplicate (and side effect of the larger issue), I'll close this in favour of #22353.