Skip to content

Fix star rating no longer updating from mod setting changes after re-entering song select#36601

Merged
peppy merged 2 commits intoppy:masterfrom
bdach:difficulty-cache-losing-tracking-over-references
Feb 6, 2026
Merged

Fix star rating no longer updating from mod setting changes after re-entering song select#36601
peppy merged 2 commits intoppy:masterfrom
bdach:difficulty-cache-losing-tracking-over-references

Conversation

@bdach
Copy link
Copy Markdown
Collaborator

@bdach bdach commented Feb 6, 2026

As "reported" in https://osu.ppy.sh/comments/3929603.

On one hand, this is firmly a second-order failure stemming from all the stupid games we're playing by making mod select overlay instances always own the game-global references to mods, but on the other hand, isn't IEquatable and its silent pervasive nature just awesome to deal with sometimes?

bdach added 2 commits February 6, 2026 10:12
…entering song select

As "reported" in https://osu.ppy.sh/comments/3929603.

On one hand, this *is* firmly a second-order failure stemming from all
the stupid games we're playing by making mod select overlay instances
always own the game-global references to mods, but on the other hand,
isn't `IEquatable` and its silent pervasive nature just *awesome*
to deal with sometimes?
@bdach bdach requested a review from peppy February 6, 2026 09:22
@bdach bdach self-assigned this Feb 6, 2026
@bdach bdach moved this from Inbox to Pending Review in osu! team task tracker Feb 6, 2026
@peppy peppy merged commit 5e01dda into ppy:master Feb 6, 2026
9 checks passed
@github-project-automation github-project-automation bot moved this from Pending Review to Done in osu! team task tracker Feb 6, 2026
@bdach bdach deleted the difficulty-cache-losing-tracking-over-references branch February 6, 2026 14:13
minetoblend pushed a commit to minetoblend/osu that referenced this pull request Feb 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants