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

Suggestion: clipping faces with OCCT #1

Closed
adam-urbanczyk opened this issue Feb 14, 2022 · 3 comments
Closed

Suggestion: clipping faces with OCCT #1

adam-urbanczyk opened this issue Feb 14, 2022 · 3 comments

Comments

@adam-urbanczyk
Copy link

You could consider clipping faces using OCCT instead of clipper (and thus operate on a more accurate representation). Clipping wires does not work, but planar faces do work (see cq.Sketch for inspiration). You'll be likely only dealing with closed wire so there is a 1-1 correspondence between faces and wires.

@voneiden
Copy link
Owner

Thanks for the tip. I imagine the primary benefit here would be preserving circular/arc features and their later conversion to G2/G3 motion commands. However clipping open wires is still necessary for creating toolpath segments, for example here's my yesterday's attempt at generating a zigzag pocket:

image

I've also thought about recovering circular motion in post by detecting sequences of three or more points that match the expected footprint of an interpolated arc/circle.

@adam-urbanczyk
Copy link
Author

I think that this use case is also supported - you can e.g. cut edges with faces or edges with edges (though in the latter case you effectively split them).

Anyways, enough of my armchair advice. Cool project!

@voneiden
Copy link
Owner

Thanks Adam, I'll definitely give that a look at some point. I'm all for prioritizing code simplicity and minimizing dependencies with this project as it has some potential to become a long term maintenance project.

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

No branches or pull requests

2 participants