fix closest_point_on_segment_xy

fixes #837

Author: Andrea Ghensi

AUTHORS.md

@@ -30,3 +30,4 @@
 - Xingxin He <<[email protected]>> [@XingxinHE](https://github.com/XingxinHE)
 - Robin Godwyll <<[email protected]>> [@robin-gdwl](https://github.com/robin-gdwl)
 - Mattis Koh <<[email protected]>> [@mattiskoh](https://github.com/mattiskoh)
+- Andrea Ghensi <<[email protected]>> [@sanzoghenzo](https://github.com/sanzoghenzo)

CHANGELOG.md

@@ -25,6 +25,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 * Changed directory where ghuser components are installed.
 * Added ghuser components directory to those removed by the `clean` task.
 * Clean up the ghuser directory before building ghuser components.
+* Fixed bug in `compas.geometry.distance.closest_point_on_segment_xy`
 
 ### Removed
 

src/compas/geometry/_core/distance.py

@@ -764,8 +764,8 @@ def closest_point_on_segment_xy(point, segment):
     d2 = distance_point_point_sqrd_xy(b, p)
     if d1 > d or d2 > d:
         if d1 < d2:
-            return a
-        return b
+            return [a[0], a[1], 0.0]
+        return [b[0], b[1], 0.0]
     return p
 
 

tests/compas/geometry/test_distance.py

@@ -0,0 +1,10 @@
+from compas.geometry import Point
+from compas.geometry import Line
+from compas.geometry import closest_point_on_segment_xy
+
+
+def test_closest_point_segment_xy():
+    point = Point(0, 0, -10)
+    line = Line([1, 1, -15], [10, 10, 20])
+    ponl = closest_point_on_segment_xy(point, line)
+    assert ponl == Point(1, 1, 0)