QRay3D

[GoogleCodeExporter]

Representa un "rayo" en el espacio tridimensional, esto es, un punto del cual 
parte una normal o dirección que traza una línea infinita imaginaria. Aporta 
funcionalidad relacionada con rayos y su interacción con otros elementos 
geométricos.

Más información en:

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

Original issue reported on code.google.com by Lince3D@gmail.com on 28 Mar 2011 at 7:36

[GoogleCodeExporter]

Empiezo con esta tarea a ver que tal.

Original comment by port...@gmail.com on 15 Jun 2011 at 3:54

• Changed state: Started

[GoogleCodeExporter]

Liberada

Original comment by port...@gmail.com on 18 Jul 2011 at 12:42

• Changed state: New

[GoogleCodeExporter]

Cogida

Original comment by chrisal...@gmail.com on 18 Jul 2011 at 6:43

[GoogleCodeExporter]

Original comment by Lince3D@gmail.com on 25 Sep 2011 at 10:31

[GoogleCodeExporter]

New owner: anderson_jag

Estado: Started

Original comment by anderson...@gmail.com on 28 Sep 2011 at 7:40

• Changed state: Started

[GoogleCodeExporter]

Tarea liberada.

Original comment by anderson...@gmail.com on 26 Oct 2011 at 4:23

[GoogleCodeExporter]

Original comment by anderson...@gmail.com on 26 Oct 2011 at 4:24

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 26 Oct 2011 at 5:28

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 2 Nov 2011 at 3:50

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión:

-¿Has hecho un plus con Intersection que recibe un QBaseLineSegment?

-"[TODO] jwladi: Esto debería ir en la clase base, es independiente de 
coordenadas.": Puedes ponerlo en QRay sí.

-Documentación de Intersection que recibe un QBasePlane: "verifies" = "verify".

-Intersection que recibe un QBaseTriangle: NumInt debe empezar por minúscula. 
numIntersections o numInter sería un nombre más apropiado, puesto que numInt 
da la sensación de referirse a un int. Es una chorrada pero lo comento.

-IntersectionPoint que recibe un QBaseOrb: Revisa las variables booleanas y los 
floats, no siguen la convención de nomenclatura. Queda más bonito así, pero 
no vamos a hacer excepciones.

-Documentación general: Por favor, en lugar de "end point" habla de "origin" 
cuando se trate de rayos.

-IntersectionPoint que recibe un QBaseTriangle: En los comentarios, sueles 
decir "ray end point". Creo que no es correcto, sería "end point of the ray" 
(bueno, según el punto anterior, "origin of the ray"). Además, hay una 
variable llamada NumInt, procede igual que en uno de los puntos anteriores por 
favor.

-Documentación de IntersectionPoint que recibe un QBaseHexahedron: "the line 
segment" = "the ray".

-Documentación de SpaceRelation: En la parte en que describes los valores 
devueltos, no estamos devolviendo un número sino un enumerado, que es muy 
distinto, aunque para C sean lo mismo. "dont" = "don't".

-SpaceRelation: Mejora este trozo de código..

if ( QFloat::IsZero(distAux) ) // Ray lies on plane
    return EQSpaceRelation::E_Contained;
if ( QFloat::IsPositive(distAux) ) // Direction vector goes to positive side
    return EQSpaceRelation::E_PositiveSide;
else // Direction vector goes to negative side
    return EQSpaceRelation::E_NegativeSide;

..poniendo un else if en lugar del segundo if.

-PointInsideTriangle y PointInsideQuadrilateral: Estás usando operadores de 
compración en el último return, en lugar de QFloat.

-Intersection que recibe 4 vectores: Estás usando el nombre NumInt también.

-IntersectionPoint que recibe 6 vectores: Estás usando el nombre NumInt 
también.

-Al final del fichero, pone __QRAY2D__ en lugar de __QRAY3D__.

Original comment by Lince3D@gmail.com on 8 Nov 2011 at 11:23

• Changed state: Started

[GoogleCodeExporter]

Original comment by jwl...@gmail.com on 9 Nov 2011 at 6:54

• Changed state: ReadyForReview

[GoogleCodeExporter]

Resultado de la revisión: Correcta.

-Documentación de SpaceRelation: Por favor, quita los números del 0 al 3 en 
los valores devueltos, es por no dar la impresión de que los valores 
enumerados corresponderán con esos números.

Un gran trabajo. En serio.

Original comment by Lince3D@gmail.com on 11 Nov 2011 at 7:05

• Changed state: Done