Issue #933

Author: QuimMoya

src/cpp/geometry/operations/boolean-utils/clip-mesh.h

@@ -82,10 +82,6 @@ namespace fuzzybools
             {
                 if (isInside2 == MeshLocation::INSIDE || isInside1 == MeshLocation::OUTSIDE)
                 {
-                    if(isInside1 == MeshLocation::INSIDE)
-                    {
-                        doit = true;
-                    }
                     // inside 2, with subtract, means don't include
                     // outside 1, with subtract, means don't include
                 }

src/cpp/geometry/operations/boolean-utils/shared-position.h

@@ -366,8 +366,7 @@ namespace fuzzybools
 
         bool IsEqualTo(const Vec3& n, double d)
         {
-            return (equals(normal,  n, toleranceVectorEquality) && equals(distance,  d, toleranceScalarEquality)) ||
-                   (equals(normal, -n, toleranceVectorEquality) && equals(distance, -d, toleranceScalarEquality));
+            return (equals(normal,  n, toleranceVectorEquality) && equals(distance,  d, toleranceScalarEquality));
         }
 
 //============================================================================================
@@ -901,6 +900,19 @@ namespace fuzzybools
                     continue;
                 }
 
+                if (isA)
+                {
+                    #ifdef CSG_DEBUG_OUTPUT
+                        DumpGeometry(geom, L"Initial_A.obj");
+                    #endif
+                }
+                else
+                {
+                    #ifdef CSG_DEBUG_OUTPUT
+                        DumpGeometry(geom, L"Initial_B.obj");
+                    #endif
+                }
+
                 auto a = geom.GetPoint(f.i0);
                 auto b = geom.GetPoint(f.i1);
                 auto c = geom.GetPoint(f.i2);
@@ -1196,6 +1208,7 @@ namespace fuzzybools
             //auto contourLoop = FindLargestEdgeLoop(projectedPoints, edges);
 
             #ifdef CSG_DEBUG_OUTPUT
+                std::vector<std::vector<glm::dvec2>> edges3DTriangles;
                 std::set<std::pair<size_t, size_t>> edgesTriangles;
                 std::set<std::pair<size_t, size_t>> finalEdgesTriangles;
             #endif
@@ -1249,6 +1262,13 @@ namespace fuzzybools
                 // TODO: why is this swapped? winding doesnt matter much, but still
                 geom.AddFace(ptB, ptA, ptC);
 
+                #ifdef CSG_DEBUG_OUTPUT
+                    edges3DTriangles.push_back({ glm::dvec2(ptA.z+ ptA.x/2, ptA.y+ ptA.x/2), glm::dvec2(ptB.z+ ptB.x/2, ptB.y+ ptB.x/2) });
+                    edges3DTriangles.push_back({ glm::dvec2(ptA.z+ ptA.x/2, ptA.y+ ptA.x/2), glm::dvec2(ptC.z+ ptC.x/2, ptC.y+ ptC.x/2) });
+                    edges3DTriangles.push_back({ glm::dvec2(ptB.z+ ptB.x/2, ptB.y+ ptB.x/2), glm::dvec2(ptC.z+ ptC.x/2, ptC.y+ ptC.x/2) });
+                    DumpSVGLines(edges3DTriangles, L"edges_tri.html");
+                #endif
+
                 #ifdef CSG_DEBUG_OUTPUT
                     finalEdgesTriangles.insert(std::make_pair(tri.vertices[0], tri.vertices[1]));
                     finalEdgesTriangles.insert(std::make_pair(tri.vertices[1], tri.vertices[2]));