[Insight-developers] CHANGES TO itkRigid3DTransform and itkRigid2DTransform

[Luis Ibanez]

Hi Hans,

The Rigid3DTransform is not intended to be used directly.

It probably should have been made an abstract class.

Calling GetParameters() and SetParameters() in the Rigid3DTransform,
should not be allowed either. (although this won't be an issue if
we make the class abstract).

The classes to use are its derives classes, that define the
specific parameterizations.

In particular:


   *   itkVersorRigid3DTransform    = Versor
   *   itkEuler3DTransform          = Euler Angles
   *   itkQuaternionRigidTransform  = Quaternion
   *   itkVersorTransform           = Versor


My suggestion would be to rather make itkRigid3DTransform an
abstract class.

Although at this point, this may not be the right backward
compatible solution....


    Luis


----------------------
Hans Johnson wrote:
> Hello All,
> 
> I just wanted to bring your attention to a bug that I’ve just addressed. 
>  All the regression tests are passing (including a new test).  The 
> changes made address the problems that I had (i.e. That GetParameters 
> always had [0,0,0] as the translation parameters).  I’d just like others 
> to review these changes to make sure that I’ve not over looked some 
> subtle inconstancies in the relationship between Offset and Translation.
> 
> http://public.kitware.com/Bug/view.php?id=8320
> 
> Thanks,
> Hans
> 
> 
> ------------------------------------------------------------------------
> 
> _______________________________________________
> Insight-developers mailing list
> Insight-developers at itk.org
> http://www.itk.org/mailman/listinfo/insight-developers