Gepetto / gepetto-viewer

Graphical Interface for Pinocchio and HPP.
BSD 2-Clause "Simplified" License
44 stars 30 forks source link

Parsing of Collada files #95

Closed jmirabel closed 5 years ago

jmirabel commented 5 years ago

Collada has a tag up_axis. Assimp seems not to read it while OSG seems to read it.

Assimp is used to parse mesh files in ROS and in hpp-fcl. It seems this is a bug in Assimp. However, it is not possible to solve this bug. There are too many people relying on the current behaviour of assimp.

I see several solutions to this recurrent issue:

jmirabel commented 5 years ago

Solved by adding a bug in GV to follow Assimp standard. PR is #96.

// FIXME: Fixes https://github.com/Gepetto/gepetto-viewer/issues/95 // The bug: Assimp seems to ignore the DAE up_axis tag. Because this // cannot be fixed in assimp without a huge impact, we make GV // compatible with assimp. // // The fix: OSG DAE plugin rotates the actual model with a root // PositionAttitudeTransform, when the DAE up axis is not Z. // We simply reset the attitude.