-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Song select performance really bad after spectating in a multi lobby #29088
Comments
I was about to make an issue for this too, I didn't know exactly it was after spectating but I noticed most times after a multi I would get horrid performance at song select, specially when hitting the random button. |
I can't reproduce anything here, and the graphs you are showing aren't really crippling slowdowns either. I've checked for leaks from multiplayer spectator and cannot find any. I've profiled performance locally and can't get a slowdown large enough to investigate further. |
Once you're in the bad state, can you create a memory dump from task manager (right click on the osu! process) then email it to me? You'll probably need to zip and upload to google drive as it may be quite big. |
Should be sent now 👌 |
One observation I've noticed is that if you don't have a lot of maps, the stuttering subsides after a few "rounds" around the beatmap carousel and the stutter isn't that bad to begin with. Osu.2024.08.27.-.12.56.39.03.webm |
Very likely closes ppy#29088. It's the only thing I could find odd in the memory dump.
Very likely closes ppy#29088. It's the only thing I could find odd in the memory dump.
There's a fix which will be included in the next release. Please check whether it fixes your issue. If not, please leave a reply here. |
Can confirm, reopening. |
Were you both testing in debug? I'm not sure that is representative of anything if yes. |
Tested with release and no difference at all. Release, debug and the current release (817.0) have the same behavior. |
I've been trying to repro this with a huge beatmap database and still can't... Confirmation that everyone that is reproing this is on windows? |
I'm on windows 11 23h2 |
Also initially tested on Windows 11, but I can also reproduce it on macOS.
That doesn't matter, I think. I only have 744 matches (beatmap difficulties), and can only reproduce if I have the |
@Joehuu and it's enough to just enter the multiplayer listing once and exit out? |
No, you have to be in |
Thanks |
Able to reproduce. Definitely GC related. Cycling carousel normally: Cycling carousel after multiplayer spectator: Comparing total genX sizes and object counts shows a large but not crippling difference (68.56mb -> 101.90mb managed / 553k -> 824k live objects). First thought was audio adjustments (each screen has muting stuff) so I attempted disabling that but it's still performing horribly. Will require some further investigation. |
@Joehuu can you try updating your .NET and reproducing on the latest HEAD? I can't seem to repro anymore... |
Never mind, fix incoming. |
Type
Performance
Bug description
Stutters when pressing f2 or causing big jumps in beatmap selection (as in going from one end of the beatmap carousel to another) after going into spectator in a multi lobby and exiting
Screenshots or videos
Toolbox.2024.07.25.-.22.39.02.32.mp4
https://discord.com/channels/188630481301012481/1097318920991559880/1266115025031004181
https://discord.com/channels/188630481301012481/1097318920991559880/1266112819791925391
Version
2024.726.0-lazer
Logs
compressed-logs.zip
The text was updated successfully, but these errors were encountered: