View previous topic :: View next topic |
Author |
Message |
vektorboson
Joined: 14 Sep 2006 Posts: 44
|
Posted: Wed Nov 07, 2007 10:56 am Post subject: Helix bug |
|
|
Hi,
I'm using my own copy of Helix and I found a bug regarding transformation between Quaternions and Matrices. (At least for me there is a bug)
Code: |
auto M = Matrix33.rotation(a, b, c);
auto Q = Quaternion(M);
auto M2 = Matrix33.rotation(Q);
assert( equal( M, M2) );
|
This assertion evaluates to false; it seems that M2 is the transpose (resp. inverse) of M. It seems to be specifically only the Quaternion->Matrix conversion. So it would be best to fix it in the Matrix33.rotation method.
Edit: Only Matrix33.rotation has this bug! (Matrix44.rotation is correct) |
|
Back to top |
|
|
baxissimo
Joined: 23 Oct 2006 Posts: 241 Location: Tokyo, Japan
|
Posted: Mon Nov 19, 2007 9:48 am Post subject: Re: Helix bug |
|
|
vektorboson wrote: | Hi,
I'm using my own copy of Helix and I found a bug regarding transformation between Quaternions and Matrices. (At least for me there is a bug)
Code: |
auto M = Matrix33.rotation(a, b, c);
auto Q = Quaternion(M);
auto M2 = Matrix33.rotation(Q);
assert( equal( M, M2) );
|
This assertion evaluates to false; it seems that M2 is the transpose (resp. inverse) of M. It seems to be specifically only the Quaternion->Matrix conversion. So it would be best to fix it in the Matrix33.rotation method.
Edit: Only Matrix33.rotation has this bug! (Matrix44.rotation is correct) |
Great. Thanks for letting me know about it. I checked in a fix to the OpenMesh Helix SVN.
Sorry for the delay in repsonse. I don't check the forums to often so the mailing list or a ping post to a pre-existing topic is the best way to reach me. Or private message. Or direct email. You get the idea. |
|
Back to top |
|
|
|
|
You cannot post new topics in this forum You cannot reply to topics in this forum You cannot edit your posts in this forum You cannot delete your posts in this forum You cannot vote in polls in this forum
|
Powered by phpBB © 2001, 2005 phpBB Group
|