search

fynv / nvidia-texture-tools

Automatically exported from code.google.com/p/nvidia-texture-tools
Other
0 stars 0 forks source link

Patch for /trunk/src/nvtt/Context.cpp #192

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
The default behavior of normal map encoding doesn't pack / unpack normals.
-> normalizeNormalMap() operates on [0;1] instead of [-1;1] ; this leads to 
wrong results.

possible optimizations :
- I think it should be only necessary if normalizeMipmaps is set.
- It might be possible to move the first img normals expansion after it is 
copied to tmp, to avoid repacking temp afterwards (l.283 in my modified file)

Original issue reported on code.google.com by pierre.m...@gmail.com on 16 Jul 2013 at 3:48

Attachments:

GoogleCodeExporter commented 9 years ago 
Ah, thanks for the patch! We don't use the InputOptions code path anymore, so I 
missed this. Your changes look good, I'll apply them as is.

Original comment by cast...@gmail.com on 19 Jul 2013 at 11:49