AreonDev / assimp-net

Managed wrapper around the assimp open asset import library
2 stars 1 forks source link

Noticed that normal maps (bump maps) are incorrectly loaded as the 'Height' type textures. #13

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Get a model with a bump/normal map.
2. Load it and observe that the normal map is of 'Height' type, rather than 
'Normals'. Note that 'Height' type should contain a displacement map if any.

What is the expected output? What do you see instead?
Expect normal maps to be loaded as Normals textures.
Normal maps are loaded as Height textures.

What version of the product are you using? On what operating system?
Well, the one before the latest. Cannot get the latest to work at the moment 
due to the Tesla problem.

Please provide any additional information below.
Attaching sample DAE model on which you can easily repro the issue.

Original issue reported on code.google.com by spershin...@gmail.com on 24 Jun 2013 at 6:29

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by nicholas.woodfield on 24 Jun 2013 at 7:00

GoogleCodeExporter commented 9 years ago
A quick look at the material code and I don't see a typo with the texture 
types. A quick google search yields some discussions on the Assimp SF forum 
(and others) that apparently Assimp uses the Height map tex type to store bump 
maps. Seems to be on the unmanaged importer end.

Original comment by nicholas.woodfield on 24 Jun 2013 at 9:01

GoogleCodeExporter commented 9 years ago
I had this thought at the back of my mind, should have checked it rather than 
posting a new issue. Thanks!
That contradicts their documentation... Strange no one has done anything about 
it.

Original comment by spershin...@gmail.com on 25 Jun 2013 at 12:41

GoogleCodeExporter commented 9 years ago
Closing this since its a cooky behavior of the assimp collada importer.

Original comment by nicholas.woodfield on 27 Jun 2013 at 3:34