Skip to content

Add plane slicer raster planner with legacy behavior#365

Merged
marip8 merged 1 commit intoros-industrial:masterfrom
marip8:feature/legacy-plane-slice-planner
Jan 8, 2026
Merged

Add plane slicer raster planner with legacy behavior#365
marip8 merged 1 commit intoros-industrial:masterfrom
marip8:feature/legacy-plane-slice-planner

Conversation

@marip8
Copy link
Member

@marip8 marip8 commented Jan 8, 2026

PR #350 revised the plane slicer raster planner to remove control over the minimum hole size, minimum segment length and point spacing in favor of performing these tasks with tool path modifiers (which all now exist). For convenience, this PR adds a new plane slicer raster planner (called PlaneSlicerLegacy) that has the same functionality as the plane slice planner before the changes introduced in #350. The new tool path modifiers are embedded directly in the planner to provide control over the min hole size, min segment length, and point spacing.

… legacy planner, including control over point spacing, min segment size, and min hole size
@marip8 marip8 force-pushed the feature/legacy-plane-slice-planner branch from 48e0de0 to cde9a15 Compare January 8, 2026 21:58
@marip8 marip8 merged commit b95bd73 into ros-industrial:master Jan 8, 2026
8 checks passed
@marip8 marip8 deleted the feature/legacy-plane-slice-planner branch January 8, 2026 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant