Skip to content
This repository has been archived by the owner on Jan 3, 2023. It is now read-only.

Added option for signed or unsigned BC5 formats #13

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

Conversation

al2950
Copy link

@al2950 al2950 commented Apr 15, 2016

No description provided.

@christop4D
Copy link

Does this work with the normalize and flip x,y operations that are implemented?
And there is also a mipmap rectification operation going on where the mipmaps of a normal map are normalized.
Can you verify this.

@al2950
Copy link
Author

al2950 commented Apr 15, 2016

Sorry, I did test those bit thoroughly enough to comment, and unfortunately I do not have access to photoshop and those who do have gone home for the weekend. I will make sure to check them on Monday/Tuesday.

However I will say I was very careful to check all bits of code that referenced "BC5" and "NORMALMAP", also the conversion to a signed format actually happens within the compression algorithm, and so the input is unchanged.

@al2950
Copy link
Author

al2950 commented Apr 22, 2016

Apologies for the delay, but I have now got one my artists to run a number of tests and I can say that the functions you mentioned still work as expected.

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

Successfully merging this pull request may close these issues.

2 participants