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

GameSettings: Disable GPU Texture Decoding when needed #13058

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Dentomologist
Copy link
Contributor

Arbitrary Mipmap Detection doesn't work when GPU Texture Decoding is enabled, so disable GPU Texture Decoding for games where the .ini enables Arbitrary Mipmap Detection.

This skips GMS.ini because that's handled in #13056.

@mbc07
Copy link
Contributor

mbc07 commented Sep 8, 2024

The change itself is fine and LGTM, but having two separate PRs handling the same thing is a bit redundant IMO, I'd suggest closing the other one and bundling the GMS.ini change on this PR as well...

@Dentomologist
Copy link
Contributor Author

I strongly prefer not to snipe someone's PR, especially when it inspired this one.

@mbc07
Copy link
Contributor

mbc07 commented Sep 8, 2024

Oh, sorry, haven't noticed you're not the author of the other PR. In that case this is fine, I guess...

@McAchi
Copy link

McAchi commented Sep 10, 2024

I've closed the PR that's only about GMS because I didn't know what I was doing and got a merge commit in the mix.
You're free to add the needed modification to GMS.ini now.

Sorry for the trouble. Thank you.

@Dentomologist Dentomologist force-pushed the gamesettings_disable_gpu_texture_decoding_when_needed branch from 301f939 to da60b38 Compare September 10, 2024 18:50
Arbitrary Mipmap Detection doesn't work when GPU Texture Decoding is
enabled, so disable GPU Texture Decoding for games where the .ini
enables Arbitrary Mipmap Detection.

Co-authored-by: jeremyie <[email protected]>
@Dentomologist Dentomologist force-pushed the gamesettings_disable_gpu_texture_decoding_when_needed branch from da60b38 to b1d54e4 Compare September 10, 2024 19:04
@Dentomologist
Copy link
Contributor Author

Sorry for the trouble myself. I've added GMS.ini and made you a coauthor for this commit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants