search

sanyaade-g2g-repos / quimeraengine

Automatically exported from code.google.com/p/quimeraengine
0 stars 1 forks source link

QPlane Extension 003 #169

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Añade un constructor a partir de 3 puntos y funcionalidad relacionada con 
transformaciones.

Esta tarea incluye el renombrado de varios métodos de QPlane, de la siguiente 
manera:

* Transform --> Translate
-Transform(const QTranslationMatrix4x3 &m)
-Transform(const QTranslationMatrix4x3 &m, QBasePlane &pOut) const
-Transform(const QTranslationMatrix4x4 &m)
-Transform(const QTranslationMatrix4x4 &m, QBasePlane &pOut)

* Transform --> Rotate
-Transform(const QRotationMatrix3x3 &m)
-Transform(const QRotationMatrix3x3 &m, QBasePlane &pOut) 
-Transform(const QQuaternion &q)
-Transform(const QQuaternion &q, QBasePlane &pOut) const

* Transform --> Scale
-Transform(const QScaleMatrix3x3 &m)
-Transform(const QScaleMatrix3x3 &m, QBasePlane &pOut) const

Más información aquí:

http://quimeraengine.googlecode.com/files/QPlane%20Extension%20003.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 4 Aug 2011 at 7:07

GoogleCodeExporter commented 9 years ago 
Status: Started, Owner: anderson_jag

Original comment by anderson...@gmail.com on 29 Aug 2011 at 5:20

GoogleCodeExporter commented 9 years ago 
Tarea liberada.

Estado: New.

Original comment by anderson...@gmail.com on 23 Sep 2011 at 4:06

GoogleCodeExporter commented 9 years ago 
La voy cogiendo :)

Original comment by jwl...@gmail.com on 17 Oct 2011 at 4:03

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 17 Oct 2011 at 7:56

GoogleCodeExporter commented 9 years ago 
Resultado de la revisión:

-Constructor que recibe 3 puntos y métodos Translate: Vamos a ofrecer las 2 
sobrecargas disponibles, una para QBaseVector3 y otra para QBaseVector4, ambas 
podrán usar internamente el método de plantilla.

-Scale: La escala se aplica multiplicando, donde 1 = 100%. El plano debería 
devolverse normalizado.

Original comment by Lince3D@gmail.com on 22 Oct 2011 at 1:19

GoogleCodeExporter commented 9 years ago 
-Falta renombrar Transform que recibe un cuaternión.

Original comment by Lince3D@gmail.com on 22 Oct 2011 at 1:22

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 22 Oct 2011 at 12:11

GoogleCodeExporter commented 9 years ago 
Resultado de la revisión: Correcta.

-El constructor que recibe 3 puntos y usa un parámetro de plantilla debería 
ser protected y tener un nombre diferente del constructor real. Por ejemplo, 
QPlaneImp, QPlaneImplementation o PlaneFrom3Points...

-He estado investigando la decisión que se tomó respecto a normalizar el 
resultado de escalar. Decidí que se dijera en la documentación que el 
resultado no estaría normalizado. Sin embargo, no veo utilidad a un plano sin 
normalizar. Lo dejaremos normalizado y si en el futuro vemos que es posible 
uasrlo en otros escenarios lo replantearemos. 
https://code.google.com/p/quimeraengine/issues/detail?id=168
Por cierto, indica que se guardará el plano normalizado allá donde lo hayas 
hecho. Es importante que el usuario sepa en todo momento lo que va a ocurrir.

Original comment by Lince3D@gmail.com on 22 Oct 2011 at 1:21