I put a function to build the equivalent quaternion in EulerMatrixZYZ. More flexibility ....<br><br><div class="gmail_quote">2009/5/4 Keren Lasker <span dir="ltr">&lt;<a href="mailto:kerenl@salilab.org">kerenl@salilab.org</a>&gt;</span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">just one comment: quaternions are more accurate than matrix multiplications, and it is a common practice to use quaternions to apply multiple rotations - so i would prefer if the internal implementation of Rotation3D would stay as it is<div>
<div></div><div class="h5"><br>
On May 4, 2009, at 5:30 PM, Daniel Russel wrote:<br>
<br>
</div></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><div><div></div><div class="h5">
I think we currently have too many way of representing matrices in the IMP API.<br>
- Rotation3D<br>
- EulerAnglesZYZ<br>
- and EulerMatrix<br>
<br>
The last doesn&#39;t make any sense as there is no reason to tie together two disparate representations. Given that one can create a Rotation3D from Euler angles I don&#39;t see any reason to expose the second. If we find rotational matrices are faster than quaternions for actually performing the rotations, then we should cache the matrix in Rotation3D rather than add another class.<br>

<br>
Comments? Justifications for having many different versions?<br></div></div>
_______________________________________________<br>
IMP-dev mailing list<br>
<a href="mailto:IMP-dev@salilab.org" target="_blank">IMP-dev@salilab.org</a><br>
<a href="https://salilab.org/mailman/listinfo/imp-dev" target="_blank">https://salilab.org/mailman/listinfo/imp-dev</a><br>
</blockquote>
<br>
_______________________________________________<br>
IMP-dev mailing list<br>
<a href="mailto:IMP-dev@salilab.org" target="_blank">IMP-dev@salilab.org</a><br>
<a href="https://salilab.org/mailman/listinfo/imp-dev" target="_blank">https://salilab.org/mailman/listinfo/imp-dev</a><br>
</blockquote></div><br>