Add test for plane boundaryBox where TopoDS_Shape is neither vertex, edge or surface

Author: mpoudot

src/Core/Geom/OCCHelper.cpp

@@ -1004,15 +1004,37 @@ computeBoundingBox(const TopoDS_Shape& shape, gp_Pnt& pmin, gp_Pnt& pmax)
 {
     Bnd_Box box;
     BRepCheck_Analyzer analyzer(shape);
+
     if (analyzer.IsValid()) {
         BRepBndLib::AddClose(shape, box);
     } else {
         BRepBndLib::AddOptimal(shape, box);
     }
 
     if (box.IsVoid())
+    {
         BRepBndLib::Add(shape, box);
+    }
 
+    TopAbs_ShapeEnum type = shape.ShapeType();
+    if (type != TopAbs_FACE && type != TopAbs_EDGE && type != TopAbs_VERTEX)
+    {
+        // On teste si la box est plane
+        // Récupérer les coins min et max
+        gp_Pnt minCorner = box.CornerMin();
+        gp_Pnt maxCorner = box.CornerMax();
+
+        // Calculer les dimensions
+        double width  = maxCorner.X() - minCorner.X();
+        double height = maxCorner.Y() - minCorner.Y();
+        double depth  = maxCorner.Z() - minCorner.Z();
+
+        // Vérifier si l'une des dimensions est nulle ou quasi-nulle
+        double tolerance =  1e-7;
+        if (width < tolerance || height < tolerance || depth < tolerance) {
+            BRepBndLib::Add(shape, box);
+        }
+    }
     double xmin, ymin, zmin, xmax, ymax, zmax;
     box.Get(xmin, ymin, zmin, xmax, ymax, zmax);
     pmin.SetCoord(xmin, ymin, zmin);

test_link/test_issue262.py

@@ -0,0 +1,28 @@
+import pyMagix3D as Mgx3D
+import math
+
+# Issue262 : Le bloc créé oar boite englobante d'un volume peut avoir de mauvaises dimensions
+# la création d'un bloc sur une sphère crée un bloc plat
+
+def assertPoint(tm, name, x, y, z):
+    p = tm.getCoord(name)
+    assert math.isclose(p.getX(), x, rel_tol=1e-7)
+    assert math.isclose(p.getY(), y, rel_tol=1e-7)
+    assert math.isclose(p.getZ(), z, rel_tol=1e-7)
+
+def test_issue262():
+    ctx = Mgx3D.getStdContext()
+    ctx.clearSession() # Clean the session after the previous test
+    gm = ctx.getGeomManager()
+    tm = ctx.getTopoManager()
+    # Création du volume Vol0000
+    gm.newSphere (Mgx3D.Point(0, 0, 0), 1, Mgx3D.Portion.ENTIER)
+    # Création d'un bloc topologique structuré sans projection (Vol0000)
+    tm.newFreeTopoOnGeometry ("Vol0000")
+    # Annulation de : Création d'un bloc topologique structuré sans projection (Vol0000)
+    ctx.undo()
+    # Création d'un bloc unitaire mis dans le groupe AAA
+    ctx.getTopoManager().newFreeBoundedTopoInGroup ("AAA", 3, ["Vol0000"])
+
+    assertPoint(tm, "Som0000", -1, -1, -1)
+    assertPoint(tm, "Som0007", 1, 1, 1)