remove rdp fallback, cgal only

Author: claude

pyproject.toml

@@ -57,7 +57,6 @@ dev = [
     "pytest>=7.0",
     "pytest-benchmark",
     "pytest-cov",
-    "rdp>=0.8",
     "ruff",
     "twine",
 ]

src/compas_slicer/post_processing/simplify_paths_rdp.py

@@ -2,36 +2,21 @@
 
 from typing import TYPE_CHECKING
 
-import numpy as np
 from compas.geometry import Point
+from compas_cgal.polylines import simplify_polylines
 from loguru import logger
 
-try:
-    import rdp as rdp_py
-except ImportError:
-    rdp_py = None
-
 if TYPE_CHECKING:
     from compas_slicer.slicers import BaseSlicer
 
 
 __all__ = ["simplify_paths_rdp"]
 
-# Check for CGAL availability at module load
-_USE_CGAL = False
-try:
-    from compas_cgal.polylines import simplify_polylines as _cgal_simplify
-
-    _USE_CGAL = True
-except ImportError:
-    _cgal_simplify = None
-
 
 def simplify_paths_rdp(slicer: BaseSlicer, threshold: float) -> None:
     """Simplify paths using the Ramer-Douglas-Peucker algorithm.
 
-    Uses CGAL native implementation if available (10-20x faster),
-    otherwise falls back to Python rdp library.
+    Uses CGAL Polyline_simplification_2 implementation.
 
     Parameters
     ----------
@@ -45,51 +30,18 @@ def simplify_paths_rdp(slicer: BaseSlicer, threshold: float) -> None:
     ----------
     https://en.wikipedia.org/wiki/Ramer-Douglas-Peucker_algorithm
     """
-    if _USE_CGAL:
-        _simplify_paths_cgal(slicer, threshold)
-    else:
-        _simplify_paths_python(slicer, threshold)
-
-
-def _simplify_paths_cgal(slicer: BaseSlicer, threshold: float) -> None:
-    """Simplify paths using CGAL Polyline_simplification_2."""
     logger.info("Paths simplification rdp (CGAL)")
     remaining_pts_num = 0
 
     for layer in slicer.layers:
         if layer.is_raft:
             continue
 
-        # Batch all paths in this layer for efficient CGAL processing
         polylines = [[[pt[0], pt[1], pt[2]] for pt in path.points] for path in layer.paths]
-        simplified = _cgal_simplify(polylines, threshold)
+        simplified = simplify_polylines(polylines, threshold)
 
         for path, pts_simplified in zip(layer.paths, simplified):
             path.points = [Point(pt[0], pt[1], pt[2]) for pt in pts_simplified]
             remaining_pts_num += len(path.points)
 
     logger.info(f"{remaining_pts_num} points remaining after simplification")
-
-
-def _simplify_paths_python(slicer: BaseSlicer, threshold: float) -> None:
-    """Simplify paths using Python rdp library."""
-    if rdp_py is None:
-        raise ImportError("rdp package required for Python fallback. Install with: pip install rdp")
-
-    logger.info("Paths simplification rdp (Python)")
-    remaining_pts_num = 0
-
-    for layer in slicer.layers:
-        if layer.is_raft:
-            continue
-
-        for path in layer.paths:
-            pts_rdp = rdp_py.rdp(np.array(path.points), epsilon=threshold)
-            path.points = [Point(pt[0], pt[1], pt[2]) for pt in pts_rdp]
-            remaining_pts_num += len(path.points)
-
-    logger.info(f"{remaining_pts_num} points remaining after simplification")
-
-
-if __name__ == "__main__":
-    pass