COLLADA2GLTF icon indicating copy to clipboard operation
COLLADA2GLTF copied to clipboard

2.0 models should not have BINORMAL

Open lasalvavida opened this issue 6 years ago • 2 comments

Currently we write out BINORMAL from COLLADA if it is present, but it is no longer a valid semantic in glTF 2.0 since it can be computed from TANGENT and NORMAL.

lasalvavida avatar Apr 09 '18 18:04 lasalvavida

This probably refers to https://github.com/KhronosGroup/COLLADA2GLTF/blob/54e9d78169bc06d13430b633ca9352293ec71364/src/COLLADA2GLTFWriter.cpp#L591 ?! (Just asking because it's still called BINORMAL there. But it might not matter, iff this case can simply be omitted...)

javagl avatar Apr 13 '18 17:04 javagl

It is to be omitted, but thanks for the correction @javagl.

lasalvavida avatar Apr 13 '18 17:04 lasalvavida