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

Add support for signed BC4 and BC5 #18

Open
RunDevelopment opened this issue Jun 12, 2024 · 3 comments
Open

Add support for signed BC4 and BC5 #18

RunDevelopment opened this issue Jun 12, 2024 · 3 comments

Comments

@RunDevelopment
Copy link

BC4 and BC5 have signed versions (BC4_SNORM and BC5S) that aren't supported right now. Please add support for them. While the signed formats are less common, they are used nonetheless, so they should be supported.

I have already implemented a BC4_SNORM decoder, so feel free to use my code.

@ScanMountGoat
Copy link
Owner

Is there a specific file you're trying to decode that doesn't work properly?

@ScanMountGoat
Copy link
Owner

The snorm variants should still work with image_dds. You may need to remap the value range of the output to convert from unsigned to signed.

@RunDevelopment
Copy link
Author

I'm sorry for not being specific enough. I meant request for signed BC4/5 to be added to bcdec_rs.

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

No branches or pull requests

2 participants