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

Bevel function for contiguous suite of edges #13

Open
wants to merge 14 commits into
base: master
Choose a base branch
from

Conversation

jimy-byerley
Copy link
Contributor

This is a function to bevel edges.
It works for a curve (a suite of edges, with no T junction) therefore passed as a [VertexID]
This function is not exactly a chamfer function: a chamfer would cut the mesh leaving the remaining faces in the exactly same position and orientation. The stitch function is already there for this purpose.
This operator is more a way to insert a chamfer-like edge, by displacing points around the edge (the neighboring faces are slightly moved, and not cutted).

I want to add a bevel function for vertices too. I will make an other PR.

@jimy-byerley
Copy link
Contributor Author

oops, the last two commits are not for bevel_curve but for bevel_vertex that is not mature yet. (I will need something like the Shape::triangulation function for it)

@asny
Copy link
Owner

asny commented Dec 9, 2019

Looks interesting! I will take a closer look at it as soon as I have time which might not be until after Christmas, sorry!

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