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

Dæmon | Make top mip-level renormalization an opt-out option #2

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

Conversation

illwieckz
Copy link

@illwieckz illwieckz commented Nov 15, 2017

Until now at Unvanquished, we used an old crunch fork that was patched by @gimhael on this purpose, he said in that time:

Some tools scale the vectors in normalmaps to maximum length for
increased precision, but that breaks reconstruction of the z part
in DXN compressed normal maps. Passing -renormalize on compression
fixes those textures now.

This PR reports the change to upstream, making it as an opt-out option to not change the default -renormalize behavior.

Users can do crunch -renormalize -rtopmip to get the behavior they need.


patch from https://github.com/DaemonEngine/crunch

- Make -renormalize work on the top mip-level too.

Some tools scale the vectors in normalmaps to maximum length for
increased precision, but that breaks reconstruction of the z part
in DXN compressed normal maps. Passing -renormalize on compression
fixes those textures now.

- Make top mip-level renormalization an opt-out option

Users can use -renormalize -rtopmip
@illwieckz illwieckz mentioned this pull request Feb 18, 2018
11 tasks
@illwieckz illwieckz changed the title Make top mip-level renormalization an opt-out option Make top mip-level renormalization an opt-out option / by DæmonEngine Nov 1, 2018
@illwieckz illwieckz changed the title Make top mip-level renormalization an opt-out option / by DæmonEngine Dæmon | Make top mip-level renormalization an opt-out option Nov 1, 2018
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

Successfully merging this pull request may close these issues.

2 participants