0027222: Empty result of making volume operation

1. BOPAlgo_MakerVolume - Taking into account the possibility of Same Domain faces
   while collecting the faces for building the solids;
2. BOPDS_DS - Making the check on coincidence of edges consistent with the intersection
   algorithm which also adds the Precision::Confusion() to the intersection tolerance;
3. Test case for the issue.
4. Adjusting test cases (improvements) to their new behavior.

diff --git a/src/BOPAlgo/BOPAlgo_MakerVolume.cxx b/src/BOPAlgo/BOPAlgo_MakerVolume.cxx
index b596db01541e33053525354b38172ef6f23b6a98..69808488c89fa4ecbdc4d6125bfc546bc4250dfd 100644 (file)
--- a/src/BOPAlgo/BOPAlgo_MakerVolume.cxx
+++ b/src/BOPAlgo/BOPAlgo_MakerVolume.cxx
@@ -200,6 +200,7 @@ void BOPAlgo_MakerVolume::CollectFaces()
   //
   Standard_Integer i, aNbShapes;
   BOPCol_ListIteratorOfListOfShape aIt;
+  BOPCol_MapOfShape aMFence;
   //
   aNbShapes = myDS->NbSourceShapes();
   for (i = 0; i < aNbShapes; ++i) {
@@ -217,7 +218,9 @@ void BOPAlgo_MakerVolume::CollectFaces()
       aIt.Initialize(aLFIm);
       for (; aIt.More(); aIt.Next()) {
         const TopoDS_Shape& aFIm = aIt.Value();
-        AddFace(aFIm, myFaces);
+        if (aMFence.Add(aFIm)) {
+          AddFace(aFIm, myFaces);
+        }
       }
     }
     else {

diff --git a/src/BOPDS/BOPDS_DS.cxx b/src/BOPDS/BOPDS_DS.cxx
index a89cad8dc1737b5d682a7d5b212ce250d852d5b9..199487f70fa2848c3e424513bc387eb4f147bae2 100644 (file)
--- a/src/BOPDS/BOPDS_DS.cxx
+++ b/src/BOPDS/BOPDS_DS.cxx
@@ -1734,7 +1734,7 @@ Standard_Boolean BOPDS_DS::CheckCoincidence
     aD=aPPC.LowerDistance();
     //
     aTol=BRep_Tool::Tolerance(aE1);
-    aTol=aTol+BRep_Tool::Tolerance(aE2);
+    aTol=aTol+BRep_Tool::Tolerance(aE2) + Precision::Confusion();
     if (aD<aTol) {
       aT2x=aPPC.LowerDistanceParameter();
       if (aT2x>aT21 && aT2x<aT22) {

diff --git a/tests/boolean/volumemaker/A3 b/tests/boolean/volumemaker/A3
index 0ca9e049d5d73fb1ce9d6f208f5e12958b0500ee..e18c15dafd929b50d13de2e4d501045e9ef12815 100644 (file)
--- a/tests/boolean/volumemaker/A3
+++ b/tests/boolean/volumemaker/A3
@@ -1,10 +1,6 @@
 # test script on make volume operation
 # plane unstable
 
-puts "TODO OCC26020 ALL: Error: bopcheck failed"
-puts "TODO ?OCC26020 ALL: Faulty shapes in variables faulty_1"
-puts "TODO OCC26020 ALL: Error : The area of result shape is"
-
 # planar face 
 plane pln_f1 0 -870 -1.3877787807814457e-014 0 1 1.1102230246251565e-016
 erase pln_f1
@@ -38,5 +34,5 @@ mkface f6 pln_f6 -1000000 1000000 -1000000 1000000
 # make volume operation 
 mkvolume result f1 f2 f3 f4 f5 f6
 
-checkprops result -s 1183220.
+checkprops result -s 1.00062e+006
 

diff --git a/tests/boolean/volumemaker/A7 b/tests/boolean/volumemaker/A7
index 8ddc568f6cdc11a91521aad5d798c60eedfd8917..8f2b4443b6db0171a4c766b58a6327f1a49afcd6 100644 (file)
--- a/tests/boolean/volumemaker/A7
+++ b/tests/boolean/volumemaker/A7
@@ -1,9 +1,6 @@
 # test script on make volume operation
 # plane unstable
 
-puts "TODO OCC26020 ALL: Error: bopcheck failed"
-puts "TODO OCC26020 ALL: Faulty shapes in variables faulty_1 to"
-
 # planar face 
 plane pln_f1 -1863.18155559 1250 -1538.57262704 1.110223024625157e-016 1 1.1102230246251563e-016
 erase pln_f1

diff --git a/tests/boolean/volumemaker/B6 b/tests/boolean/volumemaker/B6
index f3edcef2151be4fecd980b2cbcbc2c933d971458..e55a34411f02b4b7b39525b33ad9434f8c7813f4 100644 (file)
--- a/tests/boolean/volumemaker/B6
+++ b/tests/boolean/volumemaker/B6
@@ -1,6 +1,3 @@
-puts "TODO OCC26020 ALL: Faulty shapes in variables faulty_1 to faulty_"
-puts "TODO OCC26020 ALL: Error: bopcheck failed"
-
 # test script on make volume operation
 # plane
 

diff --git a/tests/boolean/volumemaker/C2 b/tests/boolean/volumemaker/C2
index 9656230095242e4a16ee6f041ad6279110eaa473..74406a53cf9e0de93ad1b296112ec0fec0838728 100644 (file)
--- a/tests/boolean/volumemaker/C2
+++ b/tests/boolean/volumemaker/C2
@@ -1,9 +1,6 @@
 # test script on make volume operation
 # plane
 
-puts "TODO ?OCC26020 ALL: Faulty shapes in variables faulty_1 to faulty_"
-puts "TODO OCC26020 ALL: Error: bopcheck failed"
-
 # planar face 
 plane pln_f1 0 840 1.2212453270876722e-014 0 -1 -1.1102230246251565e-016
 erase pln_f1

diff --git a/tests/bugs/modalg_6/bug27222 b/tests/bugs/modalg_6/bug27222
new file mode 100644 (file)
index 0000000..2861692
--- /dev/null
+++ b/tests/bugs/modalg_6/bug27222
@@ -0,0 +1,13 @@
+puts "============"
+puts "OCC27222"
+puts "============"
+puts ""
+######################################################
+# Empty result of making volume operation
+######################################################
+
+restore [locate_data_file bug27222.brep] cf
+mkvolume result cf -c
+
+checkprops result -s 838000 -v 3.5068e+007
+checknbshapes result -solid 1
\ No newline at end of file