Skip to content

Conversation

@smoogipoo
Copy link
Contributor

@smoogipoo smoogipoo commented Oct 27, 2025

Exposed as BufferedContainer.GrayscaleStrength for now, with transform support via GrayscaleTo().

I think in the future we're going to have to turn BufferedContainer into more of an "EffectContainer" with particular passes (grayscale being one of them) that can be attached, but this should be fine for now?

Using the HSP algorithm with the same parameters as defined in osu!. Maybe this can be adjustable in the future but I'm not thinking too far ahead right now.

Intended for effects such as:

2025-10-27.18-55-25.mp4

@bdach
Copy link
Collaborator

bdach commented Oct 27, 2025

Using the HSP algorithm with the same parameters as defined in osu!.

Matches this too so that's good.

Maybe we can replace that texture load hack with this shader too, not sure. Would need to see how well buffered container does there.

@smoogipoo
Copy link
Contributor Author

Probably not only because buffered container is slow :(

@bdach bdach enabled auto-merge October 27, 2025 10:56
@bdach bdach disabled auto-merge October 27, 2025 11:26
@bdach bdach merged commit 8387643 into ppy:master Oct 27, 2025
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants