-
Notifications
You must be signed in to change notification settings - Fork 5
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
Offsetting Bugs #61
Comments
I don't know if there is some correct way in cad query to create new shapes from previous ones that don't cause this behaviour or if this is indeed a bug with either cadquery or OCCT. Your feedback would be greatly appreciated! @voneiden |
|
I realized yesterday that this is actually what might be going on and I wanted to try to apply the circle offset bug patch instead of mine. I never actually thought of trying other shape combinations as that was the one I wanted to use in my designs. If that's the bug that I am encountering then I suspect that the two problematic images before want happen anymore! |
Here is the original problem as I encountered it:
If I apply my fix then everything goes back into the correct place:
The fundamental problem with my solution is that it depend on the centers not changing, which is not always the case. Some designs (see below) can create intersections when you create and an offset and those intersections need to be trimmed (hence changing the center of the wire).
When my fix is applied in the context of cq_cam then offsets that cause intersection and change the original shape will be moved into the previous center (see below).
There are however design that can still cause the issue with cq-cam:
So we need a robust way to detect when intersections happen and take that into account when correcting the offset.
The text was updated successfully, but these errors were encountered: