Rob Taglang

From #196. This will likely require some changes in OpenCOLLADA, but ideally a failure to parse should result in an error from COLLADA2GLTF. This may require calling the validator first...

This one particularly: https://github.com/assimp/assimp/blob/master/test/models/Collada/library_animation_clips.dae When multiple animations target the same node, we merge the animation keyframes and interpolate missing values, but this isn't really possible for matrix transform types. In...

Consider options for CI speed-up

1

On building with Docker: I'm only familiar with this on GitLab CI and don't know the details of how Travis and Appveyor handle this, so take it with a grain...

As in: https://github.com/KhronosGroup/glTF/pull/678 Test model: [zombie_male_a.dae.txt](https://github.com/KhronosGroup/COLLADA2GLTF/files/1936325/zombie_male_a.dae.txt)

2.0 models should not have BINORMAL

2

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...

TANGENT should be vec4 in glTF 2.0

1

Pretty self explanatory, this should be an easy fix and is a good beginner issue.

#159 added support for generating UINT32 indices when primitives exceed the capacity of UINT16. More accurately, we always generate UINT32 and then copy the indices to UINT16 if possible. The...

KHR_binary_glTF 1.0 shader encoding

1

If converting to 1.0, the shaders should be embedded in the buffer instead of the less efficient Base64 uri encoding.

See https://github.com/KhronosGroup/glTF-Sample-Models/issues/114#issuecomment-326181631 Some COLLADA files contain degenerate triangles that can be safely removed.

For example, https://github.com/KhronosGroup/glTF-Sample-Models/blob/master/2.0/CesiumMilkTruck/glTF/CesiumMilkTruck.gltf has two identical texture objects.