Re: [eigen] Rigid Transformations in eigen: discussion thread

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


2009/9/16 Adolfo Rodríguez <dofo79@xxxxxxxxx>:
>
>
> On Wed, Sep 16, 2009 at 7:50 AM, Rohit Garg <rpg.314@xxxxxxxxx> wrote:
>>
>> I have written the rigid transform class and it's unit test. The code
>> compiles, and executes correctly. The API is as close to eigen's
>> transform class as possible. I hope this is good enough to make into
>> eigen. The documentation is sketchy, but it was secondary till now.
>> The unit tests are not eigen'ish, but it is trivial to add
>> VERIFY_IS_APPROX, and I didn't want to add that to my code base.
>>
>> NOTE: If you remove the V1.normalize() and V2.normalize() calls,
>> eigen's results change. I spent a lot of brain cycles fighting this.
>> :(
>>
>> It may be a good idea to keep the axis vector in the angleaxis class,
>> and the direction vector in the parametrizedline class normalized.
>>
>> Comments/questions/cribs/criticism/commendation (!) welcome.
>
> Thanks for pushing this forward!. Definitely worth giving it a try and
> benching it with the existing Transform class (perfo-wise). Regarding the
> class name, I think it's important to clearly state that only 3D rigid
> transformations are being modeled, especially considering that the existing
> Transform class works for nD (and hence avoid confusion). Apart from that,
> and out of the top of my head, an inverse() function would be convenient,
> and maybe (?) a function for getting an equivalent Transform instance.

a) This heavily depends on fast quaternion multiplication, I think it
will likely come out somewhat behind in benches.

b) How does eigen do say, 4D rotations? I thought it did only 2D and
3D rotations. I don't know if rigid transformations make sense for 2D.
At any rate, 2D transformations are trivial.

c) Inverse would be good, I agree. You just need to do take take the
conjugate of both the quaternion parts.

d) For equivalent Transform object, rotation matrix and translation
vector are provided. Will look into generating equivalent transform
objects.

Thanks,

> Cheers and thanks again!
>
> Adolfo.
>
>>
>> --
>> Rohit Garg
>>
>> http://rpg-314.blogspot.com/
>>
>> Senior Undergraduate
>> Department of Physics
>> Indian Institute of Technology
>> Bombay
>
>
>
> --
> Adolfo Rodríguez Tsouroukdissian, Ph. D.
>
> Robotics engineer
> PAL ROBOTICS S.L
> http://www.pal-robotics.com
> Tel. +34.93.414.53.47
> Fax.+34.93.209.11.09
> AVISO DE CONFIDENCIALIDAD: Este mensaje y sus documentos adjuntos, pueden
> contener información privilegiada y/o confidencial que está dirigida
> exclusivamente a su destinatario. Si usted recibe este mensaje y no es el
> destinatario indicado, o el empleado encargado de su entrega a dicha
> persona, por favor, notifíquelo inmediatamente y remita el mensaje original
> a la dirección de correo electrónico indicada. Cualquier copia, uso o
> distribución no autorizados de esta comunicación queda estrictamente
> prohibida.
>
> CONFIDENTIALITY NOTICE: This e-mail and the accompanying document(s) may
> contain confidential information which is privileged and intended only for
> the individual or entity to whom they are addressed.  If you are not the
> intended recipient, you are hereby notified that any disclosure, copying,
> distribution or use of this e-mail and/or accompanying document(s) is
> strictly prohibited.  If you have received this e-mail in error, please
> immediately notify the sender at the above e-mail address.
>



-- 
Rohit Garg

http://rpg-314.blogspot.com/

Senior Undergraduate
Department of Physics
Indian Institute of Technology
Bombay



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/