[no subject] |

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

Also, perhaps we can keep both API ? that's all folk, what do you think ? gael. ------=_Part_10255_12608867.1219834051479 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: 7bit Content-Disposition: inline <div dir="ltr"><br>Hi list,<br><br>if we still agree that Eigen should be as close as possible to what people would write on a piece of paper, then I think Transform is poorly designed.<br><br>For instance if you want to concatenate a scale S to a transformation T you would write:<br> <br>paper: T' = T * S<br>or <br>paper: T' = S * T<br><br>while in Eigen we currently have to write:<br><br>eigen: T = T.scale(Vector3f(sx,sy,sz));<br>or<br>eigen: T = T.prescale(Vector3f(sx,sy,sz));<br><br> What about doing something similar to what I did with rotations and overloading the correct *= and * operators such that you could write:<br><br>eigen: T = T * Scale3f(sx,sy,sz);<br>eigen: T *= Scale3f(sx,sy,sz);<br> or<br> eigen: T = Scale3f(sx,sy,sz) * T;<br><br>then if you want to express:<br><br>paper: T = S * R * L;<br><br>(where L is a translation)<br>you could write:<br><br>eigen: T = Scale3f(..) * RotationType(....) * Translation3f(...);<br> <br>(with RotationType in {Quaternion, Matrix, AngleAxis, Rotation2D } )<br><br>instead of the current:<br><br>eigen: T.setIdentity();<br>eigen: T.scale(...);<br>eigen: T.rotate(...);<br>eigen: T.translate(...);<br><br> ok, actually the rotate, scale and translate methods return a reference to T, so currently you can still write:<br><br>eigen: T.setIdentity();<br> eigen: T.scale(...).rotate(...).translate(...);<br><br>that is not too bad in that case, but think about "T = R1 * L * T * R2" which involves "pre*" versions of the methods:<br><br> eigen: T.pretranslate(L).prerotate(R1).rotate(R2);<br><br>IMHO it's quite confusing because it does not give you the idea of the transformation order: "R1 * L * T * R2".<br><br>From a practical point of view we alrealy have all the RotationType, and we only need to overload the operators as well as to define two wrapper classes Scale and Translate. I'm only a bit scared about a amount of combinations to handle... though all products involving two different transformation types will generate a Transform, so that's certainely doable.<br> <br>Also, perhaps we can keep both API ? <br><br>that's all folk, what do you think ?<br><br>gael.<br><br></div> ------=_Part_10255_12608867.1219834051479--

**Messages sorted by:**[ date | thread ]- Prev by Date:
**Re: [eigen] planning** - Next by Date:
**[no subject]** - Previous by thread:
**Re: [eigen] planning** - Next by thread:
**[no subject]**

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