Bug in the face order check (#1394)

hazzlerr · web-flow · commit 2ebecc186d57 · 2025-01-13T23:21:04.000+01:00
* Fixed face order check

* Update test_planar_faces.cpp
diff --git a/src/geometry/planar.md b/src/geometry/planar.md
@@ -125,20 +125,14 @@ std::vector<std::vector<size_t>> find_faces(std::vector<Point> vertices, std::ve
                 e = e1;
             }
             std::reverse(face.begin(), face.end());
-            int sign = 0;
-            for (size_t j = 0; j < face.size(); j++) {
-                size_t j1 = (j + 1) % face.size();
-                size_t j2 = (j + 2) % face.size();
-                int64_t val = vertices[face[j]].cross(vertices[face[j1]], vertices[face[j2]]);
-                if (val > 0) {
-                    sign = 1;
-                    break;
-                } else if (val < 0) {
-                    sign = -1;
-                    break;
-                }
+            Point p1 = vertices[face[0]];
+            __int128 sum = 0;
+            for (int j = 0; j < face.size(); ++j) {
+                Point p2 = vertices[face[j]];
+                Point p3 = vertices[face[(j + 1) % face.size()]];
+                sum += (p2 - p1).cross(p3 - p2);
             }
-            if (sign <= 0) {
+            if (sum <= 0) {
                 faces.insert(faces.begin(), face);
             } else {
                 faces.emplace_back(face);
diff --git a/test/test_planar_faces.cpp b/test/test_planar_faces.cpp
@@ -93,8 +93,34 @@ void test_cycle_with_chain() {
     assert(equal_cycles(faces[1], {2, 5, 4, 5, 2, 1, 0, 3}));
 }
 
+void test_ccw_angle() {
+    std::vector<Point> p = {
+        Point(0, 2),
+        Point(1, 3),
+        Point(0, 1),
+        Point(1, 0),
+        Point(-1, 0),
+        Point(-1, 3)
+    };
+
+    std::vector<std::vector<size_t>> adj = {
+        {1, 5},
+        {0, 2},
+        {1, 3},
+        {2, 4},
+        {3, 5},
+        {0, 4}
+    };
+
+    auto faces = find_faces(p, adj);
+    assert(faces.size() == 2u);
+    assert(equal_cycles(faces[0], {0, 1, 2, 3, 4, 5}));
+    assert(equal_cycles(faces[1], {5, 4, 3, 2, 1, 0}));
+}
+
 int main() {
     test_simple();
     test_degenerate();
     test_cycle_with_chain();
+    test_ccw_angle();
 }
