Merge branch 'main' into fix-unify-cycles

CHANGELOG.md

@@ -14,6 +14,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Changed and updated the `compas_view2` examples into `compas_viewer`.
 * Changed `compas.scene.Scene` to inherent from `compas.datastructrues.Tree`.
 * Changed `compas.scene.SceneObject` to inherent from `compas.datastructrues.TreeNode`.
+* Changed `compas.geoemetry._core.predicates_3` bug fix in `is_coplanar` while loop when there are 4 points.
 * Changed to implementation of `Mesh.unify_cycles` to use the corresponding function of `compas.topology.orientation`.
 * Fixed bug in `compas.topology.orientation.unify_cycles`.
 * Fixed bug in `Mesh.thickened`.

src/compas/geometry/_core/predicates_3.py

@@ -130,7 +130,7 @@ def is_coplanar(points, tol=None):
 
     temp = points[:]
 
-    while True:
+    while len(temp) >= 3:
         a = temp.pop(0)
         b = temp.pop(0)
         c = temp.pop(0)

tests/compas/geometry/test_core_predicates_3.py

@@ -0,0 +1,13 @@
+from compas.geometry import is_coplanar
+import pytest
+
+
+def test_is_coplanar():
+
+    # 4 points that are colinear:
+    points = [[0, 0, 0], [0, 1, 0], [0, 2, 0], [0, 4, 0]]
+
+    try:
+        is_coplanar(points)
+    except IndexError:
+        pytest.fail("is_coplanar raised an IndexError")