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

Support sweeping a face of an existing shell #2098

Closed
hannobraun opened this issue Nov 16, 2023 · 1 comment · Fixed by #2121
Closed

Support sweeping a face of an existing shell #2098

hannobraun opened this issue Nov 16, 2023 · 1 comment · Fixed by #2121
Assignees
Labels
topic: core Issues relating to core geometry, operations, algorithms type: feature New features and improvements to existing features

Comments

@hannobraun
Copy link
Owner

Right now, it is possible to sweep a Face along a path, creating a Shell. This is useful for creating a new shell.

It would also be useful to extend an existing shell, by selecting one of its faces and sweeping that, creating new objects that attach to the shell. This would be very similar to the existing operation, except that the new objects should not attach the the original face, but to its neighbors, and the original face should be removed from the shell. Thus, it should be possible to reuse most of the existing code, but some refactoring will certainly be needed.

I am going to start working on this now.

@hannobraun
Copy link
Owner Author

Okay, so I was really bad about keeping this issue updated with my progress, but it's finished now 😄

Screenshot from 2023-11-30 10-18-52

It's now possible to extend a shell by sweeping one of its faces. This is a screenshot from the split model.

The API to do it is still pretty clunky, but it's a step in the right direction.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: core Issues relating to core geometry, operations, algorithms type: feature New features and improvements to existing features
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant