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

The Touch Device mod is hidden from replays #29817

Closed
ZenoG2 opened this issue Sep 9, 2024 · 2 comments · Fixed by ppy/osu-server-spectator#245
Closed

The Touch Device mod is hidden from replays #29817

ZenoG2 opened this issue Sep 9, 2024 · 2 comments · Fixed by ppy/osu-server-spectator#245

Comments

@ZenoG2
Copy link

ZenoG2 commented Sep 9, 2024

Type

Cosmetic

Bug description

Touch devices are not signaled correctly.

Screenshots or videos

Screenshot 2024-09-09 133103
Screenshot 2024-09-09 133114
Screenshot 2024-09-09 133120
Screenshot 2024-09-09 133126

Version

2024.906.2-lazer

Logs

compressed-logs.zip

@bdach bdach self-assigned this Sep 11, 2024
@bdach
Copy link
Collaborator

bdach commented Sep 11, 2024

So I presume this is happening if the user starts the play using mouse but then starts touching midway through the game because this 100% impacts only some replays. The fact of starting the touching is not conveyed to osu-server-spectator, which is responsible for recording replays, and thus is blissfully unaware of this; submission happens at the end of the score, with TD successfully present, but web will serve the replay recorded by osu-server-spectator raw which means that the information about TD is dropped from the replay.

The two obvious fixes are to either:

  • have osu-server-spectator know that mods changed (probably by sending them in the frame bundles - possibly wasteful if not done with a bit of deduplication to only send changes),
  • have osu-web correct the replay in post when serving it, similarly to how I believe it currently manufactures the replay headers for stable replays?

@ppy/team-client opinions?

@peppy
Copy link
Member

peppy commented Sep 27, 2024

probably by sending them in the frame bundles - possibly wasteful if not done with a bit of deduplication to only send changes

Frame bundles are already seldom enough that sending in them should be okay. As long as it's not some insanely huge json blob.

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.

3 participants