search

sanyaade-g2g-repos / quimeraengine

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

QPlane Extension 002 #133

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Añade funcionalidad relacionada con las transformaciones del plano.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 24 Mar 2011 at 12:03

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 3 May 2011 at 9:35

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 19 May 2011 at 8:56

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

-Transform que recibe un QScaleMatrix3x3: Habría que avisar que se 
desnormaliza el plano.

-Transform que recibe QTranslationMatrix: Se modifica D y no la normal, con lo 
que se mantiene normalizado. ¿Habría que hacer lo mismo con QScaleMatrix3x3? 
Hay que discutirlo.

-¿Por qué no usar métodos de plantilla para Transform con QTranslationMatrix?

-Transform con QTransformationMatrix4x3: Usar QFloat::_0 y _1, no afecta al 
resultado pero por estilo...

-Documentación en general: Aquellos métodos que devuelvan una copia del 
plano, deberían señalarlo de alguna manera, o puede confundir.

-¿Por qué comentas el include de QTransformationMatrix?

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

GoogleCodeExporter commented 9 years ago 
Se me había olvidado

Original comment by jwl...@gmail.com on 1 Jun 2011 at 5:12

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

-Transform que acepta un tipo de plantilla: ¿Esto se te ha olvidado borrarlo? 
Si es sólo borrarlo, da la revisión por correcta.

-Documentación de Transform que acepta un tipo de plantilla: En la 
descripción del parámetro, debería ir los tipos de matriz que pueden 
utilizarse.

Original comment by Lince3D@gmail.com on 1 Jun 2011 at 9:02

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 7 Jun 2011 at 7:11

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

Original comment by Lince3D@gmail.com on 9 Jun 2011 at 6:23