QVector3 Extension 001

[GoogleCodeExporter]

Extiende la funcionalidad de QVector3, integrándola con otras clases.

Más información en:

http://quimeraengine.googlecode.com/svn/docs/tasks/QVector3-Extension-001.pdf

Original issue reported on code.google.com by Lince3D@gmail.com on 14 Nov 2010 at 12:49

[GoogleCodeExporter]

Original comment by Lince3D@gmail.com on 25 Nov 2010 at 9:08

• Added labels: Availability-xx

[GoogleCodeExporter]

Descripción de la tarea movida a:

http://quimeraengine.googlecode.com/files/QVector3-Extension-001.pdf

Original comment by Lince3D@gmail.com on 6 Dec 2010 at 1:06

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 6 Dec 2010 at 10:16

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 7 Dec 2010 at 8:50

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-"//Monary operators": Monary = Unary.

-El método Normalize puede ser inline. Dentro, basta igualar el vector de 
entrada al vector "residente" y normalizarlo.

-CrossProduct: Puede ser inline por lo mismo que antes. Además, ¿estás 
haciendo una llamada recursiva infinita?

-Reverse: También puede ser inline.

-Transform: Está en español.

-Transform: Las sobrecargas con parámetro de salida pueden ser inline.

-"/// It's made by multiplying current vector by the matrix provided, assuming 
that current vector is (x, y, z, 1).": Esto no lo tengo claro. Un vector de 3 
dimensiones, en el espacio homogéneo (el que manejan las matrices) sólo 
indica una dirección, no un punto en el espacio (es decir, w=0). Sin embargo, 
puede ser útil tratarlo como punto... pero bueno, de momento, el que lo quiera 
tratar como punto que use un QVector4 con w=1. ¿Qué te parece (foro)?

-[TODO]: Hay que acostumbrarse a poner el nombre de quien escribió el TODO.

-Acuérdate de poner el comentario explicativo del fDot <= 1.

-Evidentemente esta tarea no pasará a Done hasta que las dependencias están 
resueltas y el código totalmente implementado, salvo excepciones de 
interbloqueo.

Original comment by Lince3D@gmail.com on 8 Dec 2010 at 8:32

• Changed state: Started

[GoogleCodeExporter]

[TODO] Waiting for Thund decision about downcasting.
[TODO] Waiting for quaternion and dual quaternion funcionalities.

Original comment by jwl...@gmail.com on 9 Dec 2010 at 6:59

• Changed state: Hinded

[GoogleCodeExporter]

[DONE]  Thund decision about downcasting. Used where necessary.

Original comment by jwl...@gmail.com on 10 Dec 2010 at 5:28

[GoogleCodeExporter]

[DONE] Quaternion and dual quaternion functionalities

Original comment by jwl...@gmail.com on 19 Dec 2010 at 4:10

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la segunda revisión:

-Falta incluir "QBaseVEctor.h".

-"QVector3 operator- () const": No es inline, por tanto debería ir en el CPP.

-"Makes a unitary copy of the current vector.": Decir copia unitaria resulta 
confuso, auqnue tú y yo sepamos lo que quieres decir, habría que explicarlo 
de otra forma.

-"Makes a reverse copy of the current vector.": Idem.

-En DotProductAngle: Por favor amplia esto "A floating point value which is the 
angle between vectors." especificando que es el ángulo más pequeño o menor a 
180º, de los 2 que se forman, o como mejor veas.

-En Transform: "Aplies" = Applies.

-En Transform: Parámetro TDQuat debe empieza con minúscula. Recuerda 
cambiarlo también en la documentación del parámetro.

-Creo que la rotación de los cuaterniones va en el sentido opuesto a las 
agujas del reloj (right-handed) pero ya lo comprobaremos.

-Ya que estás, aprovecha para poner los atributos de esta clase en el apartado 
de constantes ;)

Original comment by Lince3D@gmail.com on 21 Dec 2010 at 7:59

• Changed state: Started

[GoogleCodeExporter]

Lo de las constants no se muy bien que hacer. Yo lo hice porque en 
qquaternion lo tienes as�, pero luego en el cpp lo tienes en attributes 
inicialization, habr�a que cambiarlo a constants inicialization?

Otra cosa es que tu tienes un apartado friends en el .h que gran parte  de 
las clases no tiene, creo, pq me parece que lo incorporaste m�s tarde. En el 
.cpp no hay ese apartado, igual habr�a que ponerlo al principio tb.

Original comment by jwl...@gmail.com on 21 Dec 2010 at 9:27

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

Sólo antes de subir corrige esto:

-En la documentación de Reverse y Normalize: "a out" = "an output".

Original comment by Lince3D@gmail.com on 23 Dec 2010 at 10:07

• Changed state: Done