indiantarget / quimeraengine

0 stars 0 forks source link

Adaptar métodos a política de no normalización #168

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Se ha tomado la decisión de no incluir la normalización de parámetros de 
entrada o resultados de operaciones en el interior de las propias operaciones. 
Esta tarea consiste en avisar al usuario de esta circunstancia, eliminar 
aquellas normalizaciones existentes en la actualidad y realizar algunas 
comprobaciones:

-QPlane::InetrsectionPoint que recibe planos: Si es necesario que los planos de 
entrada estén normalizados (preguntar al autor), entonces avisar en al 
documentación y no normalizar las entradas.

-QPlane::Transform que recibe matriz de escala: El valor guardado no se 
normaliza. Avisar en la documentación.

-QRay::GetPoint: Se asume que el rayo está normalizado. Avisar en la 
documentación.

-QRay::Constructor que recibe un punto y una dirección: Indicar que la 
dirección de entrada debería estar noamalizada. No se normaliza nada.

-QPlane::SpaceRelation: Eliminar la normalización de los planos. Avisar en la 
documentación de que tanto el plano de entrada como el residente deben estar 
normalizados.

-QQuaternion::Constructor que recibe 3 ángulos: Indicar en la documentación 
que se guardará un cuaternión normalizado. Asegurarse de que efectivamente lo 
que se guarda está normalizado.

-QQuaternion::Constructores(dos) que reciben un eje y un ángulo: Indicar en la 
documentación que se guardará un cuaternión normalizado. Asegurarse de que 
efectivamente lo que se guarda está normalizado.

-QQuaternion::Slerp: Eliminar la normalización. Avisar en la documentación de 
que tanto el cuaternión de entrada como el residente deberían estar 
normalizados. Comprobar que se devuelve un cuaternión normalizado y avisarlo 
en la documentación.

-QRotationMatrix3x3::Constructor que recibe un eje y un ángulo: Eliminar la 
normalización. Avisar en la documentación de que el eje debe estar 
normalizado.

-QDualQuaternion::Constructor que recibe un vector y un cuaternión: Avisar en 
la documentación de que se esperan las entradas normalizadas.

-QDualQuaternion::Constructor que recibe un cuaternión y un vector: Avisar en 
la documentación de que se esperan las entradas normalizadas.

-Eliminar los métodos: QQuaternion::UnitReverse y QQuaternion::UnitSlerp.

Original issue reported on code.google.com by Lince3D@gmail.com on 25 Jul 2011 at 9:00

GoogleCodeExporter commented 9 years ago
Foro de discusión: 
http://kinesisforum.elalumbramiento.org/viewtopic.php?f=10&t=308&p=4203#p4203

Original comment by Lince3D@gmail.com on 25 Jul 2011 at 9:03

GoogleCodeExporter commented 9 years ago

Original comment by jwl...@gmail.com on 24 Nov 2011 at 9:56

GoogleCodeExporter commented 9 years ago
Enviado QPlane.

Original comment by jwl...@gmail.com on 25 Nov 2011 at 10:58

GoogleCodeExporter commented 9 years ago
Puedes subir QPlane.

Original comment by Lince3D@gmail.com on 29 Nov 2011 at 8:03

GoogleCodeExporter commented 9 years ago
Subido QPlane y afecciones. Ver revision r896 para más detalles.

Original comment by jwl...@gmail.com on 29 Nov 2011 at 10:37

GoogleCodeExporter commented 9 years ago
Enviado QRay

Original comment by jwl...@gmail.com on 30 Nov 2011 at 2:51

GoogleCodeExporter commented 9 years ago
Puedes subir QRay y relacionados.

Original comment by Lince3D@gmail.com on 3 Dec 2011 at 1:35

GoogleCodeExporter commented 9 years ago
Subido QRay. Ver revision r899 para más detalles.

Original comment by jwl...@gmail.com on 3 Dec 2011 at 7:38

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

-Documentación en QDualQuaternion: "to represents" = "to represent".

-Documentación en QQuaternion: Creo que se dice denormalized, no unnormalized. 
"This method offers a" prefiero que "offers" = "produces".
Unit quaternion está bien dicho, aunque da igual si lo cambias por normalized.

Original comment by Lince3D@gmail.com on 5 Dec 2011 at 12:30

GoogleCodeExporter commented 9 years ago
Subido resto de clases. Ver revisión r900 para más detalles.

Original comment by jwl...@gmail.com on 6 Dec 2011 at 8:08