search

dyrks1991 / quimeraengine

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

QDualQuaternion #70

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Representa un cuaternión dual, incluyendo funcionalidad entre objetos de la 
misma clase y de clases de las que depende.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 11 Dec 2010 at 11:58

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 12 Dec 2010 at 9:41

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 12 Dec 2010 at 9:42

GoogleCodeExporter commented 8 years ago

Original comment by jwl...@gmail.com on 27 Dec 2010 at 11:04

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

-Falta la documentación de clase de QDualQuaternion (cópialo de donde sea si 
quieres y pones la referencia).

-Documentación de constructores que aceptan un QBaseQuaternion y un 
VectorType: "taking in account" = "taking into account".

-Documentación de operator*: "by the left" no estoy muy seguro de que sea 
así, me suena más "from left to right" o al contrario, tendría que mirarlo a 
fondo.

-En operator*: "/// [TODO] jwladi: What's the matter if vector is v(x, y, z, 
1)?": Esto no recuerdo lo que era.

-Documentación de ResetToIdentity: "reseted" = "reset".

-Documentación de GetNonDualLength: "a interpolated" = "an interpolated".

-Transform: Falta hacer const al parámetro de entrada.

-Documentación de TransformTranslationFirst: "Regular quaternion which defines 
the rotation. It must be a QVector3 or a QVector4." copypaste fail :P

-Documentación de Lerp: "beteewn" = "between". 
 Además, dices que es spherical, cuando es lineal. ¿No? 
 "a interpolation" = "an interpolation". 
 "necessary apply" = "necessary to apply"

-Lerp con parámetro de salida: Falta poner const al parámetro de entrada y 
poner el [OUT] al parámetro de salida.

-Si ATTRIBUTES está vacío, quitarlo.

-Chorrada: Poner una línea en blanco después de METHODS en el CPP.

Original comment by Lince3D@gmail.com on 31 Dec 2010 at 4:19

GoogleCodeExporter commented 8 years ago

Original comment by Lince3D@gmail.com on 3 Jan 2011 at 9:47

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

Original comment by Lince3D@gmail.com on 3 Jan 2011 at 10:03