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

Timing: Reassess "platform specific" audio offset #24625

Closed
5 tasks done
Tracked by #24811
peppy opened this issue Aug 23, 2023 · 3 comments · May be fixed by #26154
Closed
5 tasks done
Tracked by #24811

Timing: Reassess "platform specific" audio offset #24625

peppy opened this issue Aug 23, 2023 · 3 comments · May be fixed by #26154

Comments

@peppy
Copy link
Member

peppy commented Aug 23, 2023

A platform offset of 24ms 15ms is applied, currently only on windows. This is intended to match osu!stable, which originally added this to fix BASS reporting current time with an unexpected offset (this was potentially fixed at bass' end when switching from DirectAudio to WASAPI, but still exists as it resulted in the offset being baked into beatmaps).

As I mentioned here, this probably needs to be applied to all platforms instead.

We should also quadruple confirm this is implemented correctly, especially in flows of

  • Gameplay
  • Replay playback
  • Actually getting the correct playback offset from BASS on all platforms
  • Check special cases in legacy decode/encode processes

There are some other related issues, but I'm opening a fresh one to track this as I believe we want to be sure that everything is okay before we get scoring online. It would suck if we started to persist replays with incorrect timing, for instance.

Task list includes all relevant issues, including rate adjust causing incorrect offset application.

Tasks

Preview Give feedback
  1. area:editor audio priority:1
  2. area:gameplay
    peppy
  3. area:editor audio
    peppy
  4. area:gameplay audio
  5. area:gameplay
@peppy peppy self-assigned this Aug 23, 2023
@peppy peppy moved this to Needs implementation in Path to osu!(lazer) ranked play Aug 23, 2023
@peppy peppy changed the title Reassess platform audio offset Reassess "platform specific" audio offset Aug 23, 2023
@longnguyen2004
Copy link

Will this mean that lazer will also have the same wrong offset issue as stable? At least for the couple maps I've tested, lazer and Quaver are having the correct offset relative to the song file. If any offset changes were to be applied, it shouldn't affect the waveform.

Map in question if you want to investigate: https://osu.ppy.sh/beatmapsets/1509132#mania/3253011
Offset on this is correct relative to waveform, so whatever offset changes applied shouldn't affect it.

@peppy
Copy link
Member Author

peppy commented Sep 5, 2023

The goal is to make things match how they should be. I can't really give more information than that.

Right now we don't really need feedback on this issue. It requires investigation by a developer who has knowledge of historical offsets in osu!'s past.

@peppy
Copy link
Member Author

peppy commented Dec 27, 2023

In the last few days I've reassessed the global offset, and it still seems somewhat correct. Also, at least for me, having 0ms offset on macOS is better than applying the 15ms. I can't say the same about linux, but for now I don't think there is anything actionable here. Will revisit at a later date next year.

@peppy peppy closed this as completed Dec 27, 2023
IepIweidieng added a commit to IepIweidieng/osu2tja that referenced this issue Nov 23, 2024
IepIweidieng added a commit to IepIweidieng/osu2tja that referenced this issue Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants