Differences

This shows you the differences between two versions of the page.

--- commit:2018:03_09 [2018/03/09 16:00] – papeleux
+++ commit:2018:03_09 [2018/03/09 16:04] (current) – papeleux
@@ Line 14: / Line 14: @@
   * Attention, il reste un soucis sur les QuadSide dont les BBoxes sont calculées sur base des 4 points, et le projectionOperator est le SurfaceOperator générique (d'où la copie de la BBox QuadSide vers la surface sous-jascente)
   * Attention2 : les BBoxes sont aussi utilisées dans le remaillage (mais de manière moins claire) à travers les fonctions "insideForConvexe", "preCompute2BoundingBox",...
-  * Attention 3 : Un certain nombre d'opérations géométrique m'ont interpellé en tombant dessus et une grosse remise à plat serait bénéfique à la lisibilité du code. Des refactoring à prévoir serait :
+  * Attention 3 : l'interaction entre BBox et les méthodes d'arbre WithBoundaryVolume ne me semblent pas nécessairement très claire (=> si vous observez un soucis, n'hésitez pas)
+  * To Be Continued : Un certain nombre d'opérations géométrique m'ont interpellé en tombant dessus et une grosse remise à plat serait bénéfique à la lisibilité du code. Des refactoring à prévoir serait :
     * d'implémenter des opérateurs de projection optimisé pour les faces maillées (QuadSide)
     * supprimer la couche "ProjectionOperator" en créant un "ProjectionData" reprenant les paramètres de projection et en déplacant les fonctions de projection dans les classes géométriques