<br>Hi Ziv,<br><br>Again, the problem with setting them separately is that the time stamp is shared.<br><br>The second setting will override the time stamp of the first setting and the Transform<br>will end up with a Translation component that does not match in time to the Rotation

<br>component.<br><br>E.g. the Translation component may even have expired, and by setting only the <br>Rotation we will extend its validity time artificially.  It is like making a sandwich with <br>fresh cheese and an expired ham.... the cheese may be good, but the total sandwich 

has gone bad anyways. If the pivot calibration is computing both components separately, then they should only  be stored in an igstkTransform when both Translation and Rotation are available, and that should be done using the SetRotationAndTranslation() method.

<br><br><br>   Luis<br><br><br>----------------------------------------------------------------------------<br><div><span class="gmail_quote">On 8/6/07, <b class="gmail_sendername">Ziv Yaniv</b> <<a href="mailto:zivy@isis.georgetown.edu">

zivy@isis.georgetown.edu</a>> wrote:</span><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><p>Hi Luis,</p>

Setting rotation and translation separately without resetting the transformation is reasonable behavior. Pivot calibration for example only sets the translational part of a transformation. The user can set the rotational part after they get the transformation from the pivot calibration according to prior knowledge (what Patrick is doing right now). Of course there are many ways to do this (composing separate rotation and translation transformations to get a single transform, or setting the transformation only when you have both rotation and translation in hand).

</p>

<p>In any case I view this as an arbitrary choice and my only preference is that the documentation and implementation match.</p>

<p><br>             Ziv</p><div><span class="e" id="q_1143c80453a2db05_1"><br><br><br>-----Original Message-----<br>From: "Luis Ibanez" <<a href="mailto:luis.ibanez@kitware.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

luis.ibanez@kitware.com</a>><br>Sent 8/6/2007 12:41:56 PM<br>To: "'Ziv Yaniv'" <<a href="mailto:zivy@isis.imac.georgetown.edu" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">

zivy@isis.imac.georgetown.edu</a>>, "IGSTK Developers" <<a href="mailto:igstk-developers@public.kitware.com" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">igstk-developers@public.kitware.com

</a>><br>Subject: igstkTransfrom documentation update & Style problems<br><br></span></div><p></p><div><span class="e" id="q_1143c80453a2db05_3"><pre>Hi Ziv,<br><br>Thanks for updating the documentation of the Transform class:

<br><a href="http://public.kitware.com/cgi-bin/viewcvs.cgi/Source/igstkTransform.h?root=IGSTK&sortby=date&r2=1.20&r1=1.19" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://public.kitware.com/cgi-bin/viewcvs.cgi/Source/igstkTransform.h?root=IGSTK&sortby=date&r2=1.20&r1=1.19

</a>

Unfortunately they introduced several coding style violations.

<a href="http://public.kitware.com/IGSTK/Sites/insight.journal.kitware/KWStyle/20070806-0430-Nightly/BuildError.html" target="_blank" onclick="return top.js.OpenExtLink(window,event,this)">http://public.kitware.com/IGSTK/Sites/insight.journal.kitware/KWStyle/20070806-0430-Nightly/BuildError.html

</a>

Also, I'm not sure that the right thing was to modify the documentation.

It seems that the SetRotation method actually has the bug of not setting

the translation to identity, and the SetTranslation() method has the bug

of not setting the rotation to identity.

This should be necessary because the transform has a single TimeStamp,

and we assume that it is the same for both Translation and Rotation.

I'll suggest that we put the documentation back, and fix the bug in

the implementation of the methods.

Do you see a reason why not initialized Translation / Rotation may

bring any advantage ?

    Thanks

       Luis

</pre></span></div></blockquote></div><br>