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

Joined edges: new "J" edge assignment #26

Closed
edemaine opened this issue Jan 27, 2020 · 0 comments · Fixed by #30
Closed

Joined edges: new "J" edge assignment #26

edemaine opened this issue Jan 27, 2020 · 0 comments · Fixed by #30
Labels

Comments

@edemaine
Copy link
Owner

edemaine commented Jan 27, 2020

FOLD requires faces to be split into subfaces for a few reasons:

  • To specify face ordering information (two coplanar nonconvex faces can be ordered one way in one part and the other way in another part)
  • To represent faces with holes

We want to keep track of these subfaces (faces in the FOLD structure) being part of a single intended/represented face ("superface"). The proposed simple way is to have a new edge assignment type, "J", for joined edge. F is an unfolded crease, and is meaningful from an origami perspective, whereas J edges are just for representation purposes.

Then we'll want a function that constructs the boundaries of a superface, by DFSing over J edges. Here we can get multiple boundaries, allowing for superfaces with holes. If the superface happens to be planar, we can characterize one as exterior and the others as holes. This effectively solves #21 in a simple way, so will likely replace #21.

Incidentally, J would be the natural assignment for triangulation edges in Origami Simulator (both for inputting manual triangulation and outputting with triangulation) — the equivalent of yellow lines in its SVG input format.

(discussion with @estollnitz, @origamimagiro, @amandaghassaei)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant