Hi Panagiotis, The main.cpp perfectly illustrates the bug, and the fact that the patch fixes it too. The patch itself looks nicely minimal and complete to me, so is now merged and submitted to SVN/trunk.
Cheers, Robert. On Wed, Oct 15, 2008 at 1:55 PM, Panagiotis Koutsourakis <[EMAIL PROTECTED]> wrote: > Hello, > > We are using Open Scene Graph for an application and we need COLLADA > support. While testing the pluggin we found a small bug and we are > submitting a patch. > > The first attachment is a small program that creates a scene with two > pyramids, transformed by two instances of > osg::PositionAttitudeTransform. One of them is rotated 90 degrees in > the X axis, and the scene is exported both in the native OSG (.osg) > and COLLADA (.dae) formats. In the first case the rotated pyramid is > displayed correctly whereas in the second the pyramid seems not to be > rotated. > > In the COLLADA 1.4.1 specification (found at > http://www.khronos.org/collada/) it is specified that > "The <rotate> element contains a list of four floating-point values > [...] followed by an angle in degrees" but the plugin seems to write > the value in radians. > > The problem seems to be in the method daeWriter::apply() that seems to > be writing the angle value in radians to the COLLADA file. The patch > can be found in the second attachment and is simply a call to > RadiansToDegrees wrapped around the angle. > > We are not really familiar with the code and so the patch might be > wrong, but it seems to work for us. > > Thank you, > > Loic Simon, > Panagiotis Koutsourakis, > Olivier Teboul. > > _______________________________________________ > osg-submissions mailing list > [email protected] > http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org > > _______________________________________________ osg-submissions mailing list [email protected] http://lists.openscenegraph.org/listinfo.cgi/osg-submissions-openscenegraph.org
