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

DXT5: Fix decoding of alpha channel for textures with a non-mod-4 width. #18554

Merged
merged 2 commits into from
Dec 15, 2023

Conversation

hrydgard
Copy link
Owner

@hrydgard hrydgard commented Dec 15, 2023

Had to step through the DXT5 decoder to see it.. Silly logical mistake.

Fixes #18108

@hrydgard hrydgard added the GE emulation Backend-independent GPU issues label Dec 15, 2023
@hrydgard hrydgard added this to the v1.17.0 milestone Dec 15, 2023
@hrydgard hrydgard merged commit dd4eed1 into master Dec 15, 2023
18 checks passed
@hrydgard hrydgard deleted the tiger-woods-texture branch December 15, 2023 10:12
@unknownbrackets
Copy link
Collaborator

I still haven't done full proper tests of dimensions with DXT, including bufw. Wasn't sure if the lower resolutions were actually "supported".

-[Unknown]

@hrydgard
Copy link
Owner Author

They kind of have to be for the smallest mipmaps, and are on every other GPU that supports DXT so seems likely - but indeed, a test would be best. But the fact that games use 2x32 textures also confirms it, really.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GE emulation Backend-independent GPU issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tiger Woods PGA Tour 10 - strange striped backgrounds in UI windows (DXT5 issue?)
2 participants