Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

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

Editor hit sound lag at 75%-100% playback speed #20293

Closed
Tracked by #24625
19peytonsmith opened this issue Sep 14, 2022 · 3 comments
Closed
Tracked by #24625

Editor hit sound lag at 75%-100% playback speed #20293

19peytonsmith opened this issue Sep 14, 2022 · 3 comments
Labels
area:editor audio priority:1 Very important. Feels bad without fix. Affects the majority of users.

Comments

@19peytonsmith
Copy link

Type

Performance

Bug description

I know it isn't a timing issue because at 25%-50% it sounds normal, the hitsounds follow according to the beat, but it seems that for the higher playback speeds the hitsounds are a bit delayed. I hope the video properly demonstrates what the issue is, you should be able to hear the delay.

Screenshots or videos

New.video.mp4

Version

2022.911.0-lazer

Logs

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

@peppy
Copy link
Member

peppy commented Sep 14, 2022

I brought this up recently, but I don't think I ended up opening an issue for it (may have gotten stuck in my personal TODO list).

The reasoning for this is that osu-stable applies an automatic offset when speed is adjusted to make up for inaccuracies in bass timekeeping (or otherwise).

This has not yet been implemented in lazer.

@peppy peppy added audio priority:1 Very important. Feels bad without fix. Affects the majority of users. labels Sep 14, 2022
@peppy
Copy link
Member

peppy commented Sep 14, 2022

Stable code for reference:

JetBrains Rider 2022-09-14 at 04 13 59

Of importance, the platform, beatmap and "windows" (aka usingWasapi in screenshot) are applied less as the playback rate is reduced, and increased as it is increased. This is done in the last line in the screenshot.

When reimplementing, care should be taken to check whether:

  • This applies across all OSes
  • This applies to frequency adjustments or only "tempo" adjustments

@peppy
Copy link
Member

peppy commented Dec 26, 2023

To clarify on my own response here: the offset in stable is specific to time stretching. In the editor, we are doing frequency modulation, which doesn't require this offset. So this issue is a bit off.

Moving to discussion for now, as I can't personally repro.

@ppy ppy locked and limited conversation to collaborators Dec 26, 2023
@peppy peppy converted this issue into discussion #26133 Dec 26, 2023
@peppy peppy changed the title Hitsound lag at 75%-100% playback speed Editor hit sound lag at 75%-100% playback speed Dec 26, 2023

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
area:editor audio priority:1 Very important. Feels bad without fix. Affects the majority of users.
Projects
None yet
Development

No branches or pull requests

2 participants