Re: [eigen] Re: proposal: call the Geometry module experimental |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen Archives
]
- To: eigen@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [eigen] Re: proposal: call the Geometry module experimental
- From: Benoit Jacob <jacob.benoit.1@xxxxxxxxx>
- Date: Fri, 23 Jan 2009 20:07:22 +0100
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=WqU4MA1RC63RF4mGAzUezHa28wHKR2DqkYIA0PqZUHE=; b=aZKkC9gHQ0DvSPktwkpthue9W7aSo8vFt7xjJ2rvrQe68BT1oAbC/FBkspfSxaI4C8 Cl6FMjQp0ruyDxN+DjHSfj6PMBbLOUGuDCO7T/TElXrWC5K+3XaWh5H5B1MEqTsg3FYi WnUKdTI+lQz5kETS4IDgblT09mrk2tyDAGE6U=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=Uw2lHHALGdP3JJzRxfJlqpqoWr4zWgUPLBF0tyGugh71E60Dg9e4zHyPlvB8c/2/iy gmFBfVMv1edtCTonKmKLHGzN9IfCuSoiE5Kpc7cMPKN1WNeVaTvRfpNX32qCzrxFZ17J vK27Muqg9YDwvjMma/99YFn+0BWM1VCXPiCew=
2009/1/23 Gael Guennebaud <gael.guennebaud@xxxxxxxxx>:
> yes I agree the current Scaling is somewhat limited. What about the
> following proposal:
>
> 1 - rename Translation TranslationProxy and slit Scaling in
> UniformScaling, AxisAlignedScaling and GenericScaling.
I'm not favorable to GenericScaling. What is a GenericScaling? it's
just a self-adjoint matrix. The two notions are equivalent (hint:
diagonalize your selfadjoint matrix, now it looks like a scaling...)
If people want to multiply a transform by a GenericScaling they should
just multiply by that matrix.
> 2 - Add the global functions Translation and Scaling which would
> auto-magically instanciate the correct proxy type.
Aaaah this is the definitive solution. Decouple the actual class from
the name that the user sees.
Then if you add the Proxy suffix for translation for coherence we
should add it to the scaling types too?
Also:
Can a Native English Speaker (tm) here tell us how good/bad
AxisAlignedScaling sounds? The same usage of the word Aligned can also
be found in our AlignedBox class. It is meant as "oriented along the
standard axes". Is that correct?
Can we say that the very existence of this discussion is a good reason
to mark the whole Geometry module as experimental?
(Still doing our best to preserve compatibility for existing projects) ?
Cheers,
Benoit