QTransformationMatrix

[GoogleCodeExporter]

Representa una matriz de 4 filas y 3 ó 4 columnas que simboliza una 
transformación compuesta por posición, rotación y escala, en orden 
indeterminado. La razón de que existan matrices de transformación a parte de 
las matrices normales se debe a que las matrices normales son genéricas, se 
ciñen a las operaciones aritmáticas de matrices matemáticas, mientras que 
las de transformación incluyen optimizaciones y se limitan a realizar 
únicamente las operaciones que se sabe que afectan al resultado. Son más 
óptimas y añaden funcionalidad de otro contexto.

Más información en:

http://quimeraengine.googlecode.com/files/QTransformationMatrix.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 14 Dec 2010 at 10:56

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 12 Jan 2011 at 7:27

• Changed state: Started

[GoogleCodeExporter]

[deleted comment]

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 21 Jan 2011 at 2:26

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: 

-Documentación de clase: "composed by a scale" = "composed of a scale". "a 
rotation and a transformation" = "a rotation and a translation". "we trate it 
as" = "we treat it as".

-El primer TYPEDEFS sobra.

-Documentación cosntructor que acepta una matriz: "The matrix in which we want 
the resident matrix to be based" ¿no sería "The matrix which we want the 
resident matrix to be based on"? No estoy seguro.

-Documentación constructor que acepta 11 floats: "displazament" = 
"displacement". "4d" = "4D" (por ponerlo igual en todas partes).

-¿Podrías hacer que el constructor que acepta 2 vectores y un cuaternión 
llamase al que acepta 11 floats? Creo además que ninguno de los dos va a ser 
considerado como inline por el compilador así que igual el modificador sobra...

-Documentación de operator* y operator*=: "Since both are transformation 
matrix" = "Since both are transformation matrices".

-Documentación de operator* (returns): "template parameter that resident 
matrix" = "template parameter as resident matrix".

-Documentación de GetDeterminant: "as if it were a 3x3 matrix" = "as if it was 
a 3x3 matrix".

-Documentación de GetDeterminant (returns): "Floating point value which is the 
result of the determinant" no sería el resultado del determinante, sería el 
determinante mismo.

-Documentación de GetScale: "Extracts the scale factors from the matrix. Since 
".

-Documentación de GetRotation: "[DOC] Quaternion where to ".

-Documentación de SwitchHandConvention: "Turns de hand convention into" = 
"Turns the hand convention into" . "left-hand convention as default" = 
"left-hand convention by default".

-SwitchHandConvention: "vInvScale.x = QFloat::_1/vScale.x;" podrías hacer que 
vInvScale se inicialice a (1,1,1) y luego usar el operador /=.

-SwitchHandConvention con parámetro de salida: Podrías hacer "m = *this;" y 
leugo "m.SwitchHandConvention();".

-ToRotationMatrix3x3: El modificador private se usará en situaciones 
concretas, de momento déjalo en protected. Es preferible usar un vector en 
lugar de "float_q fScaleX, fScaleY, fScaleZ;" y seguir la recomendación de 
más arriba.

-Documentación de ToRotation: "Function to converts" = "Function to convert".

-CONSTANTS INITIALIZACION: Alargar el "subrayado".

-Definición de la constante Identity: Sobra el modificador static y falta el 
espacio de nombres QTransformationMatrix<MatrixType>::.

Original comment by Lince3D@gmail.com on 21 Jan 2011 at 7:19

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 23 Jan 2011 at 6:51

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de ToRotation: "Function to converts" = "Function to convert".

Original comment by Lince3D@gmail.com on 23 Jan 2011 at 11:14

• Changed state: Done