-
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
Support point and segment extrusion #1245
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking at bend made me think again about how to flatten out bent wood shapes and I did some research on how other CAD programs do it and it seems like for the most part they are doing a conversion to mesh and then flattening the mesh
@@ -13,7 +13,7 @@ See: [remesh](#https://raw.githubusercontent.com/jsxcad/JSxCAD/master/nb/api/rem | |||
`; | |||
|
|||
Box(40, 1, 5) | |||
.and(bend(20)) | |||
.And(y(20), bend(20)) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does capital And do anything different?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The capital forms only use the input shape for context.
So this produces two versions of Box(40, 1, 5) -- one translated, and one bent.
The lowercase form would produce three versions -- the original, the translated, and the bent.
@@ -3,49 +3,128 @@ | |||
#include "demesh_util.h" | |||
#include "point_util.h" | |||
|
|||
static int Bend(Geometry* geometry, double reference_radius) { | |||
static int Bend(Geometry* geometry, double reference_radius, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What does bend do again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Applies a uniform curvature to the shape.
Effectively placing it along the perimeter of a circle of the given radius.
It's not quite what we generally want, though.
No description provided.