Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Raster w.r.t. a specified direction #121

Merged
merged 3 commits into from
Dec 11, 2020

Conversation

marip8
Copy link
Member

@marip8 marip8 commented Dec 11, 2020

The current surface walk tool path planner does not properly implement the specification of raster direction with respect to an axis defined in the coordinate system of the mesh. This PR eliminates the raster_wrt_global_axes_flag in favor of utilizing the cut_direction parameter. If any element of cut_direction is non-zero, it will be used as the raster direction.

Under the hood, the cut_direction vector is projected onto the plane defined by the average normal of the mesh. The projected vector then becomes the raster axis, and the average normal becomes the axis about which the raster axis can be rotated by the specified rotation offset.

Examples

Cut Direction = [-1, 1, 0]
Screenshot from 2020-12-11 13-05-28

Cut direction = [1, 1, 0]
Screenshot from 2020-12-11 13-11-01

Addresses #110
Merge after #119

@marip8 marip8 changed the title Fix/raster wrt global axes Raster w.r.t. a specified direction Dec 11, 2020
@marip8 marip8 force-pushed the fix/raster_wrt_global_axes branch from 4df2b87 to 8302955 Compare December 11, 2020 20:08
@marip8 marip8 merged commit f250219 into ros-industrial:master Dec 11, 2020
@marip8 marip8 deleted the fix/raster_wrt_global_axes branch December 11, 2020 20:16
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.

2 participants