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

Clipping feature is implemented #188

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

Conversation

sariug
Copy link
Contributor

@sariug sariug commented Mar 9, 2020

Hello Guillaume,

As I stated in #186 I implemented clipping feature. Here is the screenshots:
image
image
image
image

I tried following the documentation and coding style as much as possible.
Let me know for open issues, as well as bugs you think.

Best regards,

@guillaumechereau
Copy link
Owner

Thanks, I'll have a look as soon as I can!

@guillaumechereau
Copy link
Owner

I just had a look. What is the use case for it that cannot be done with a selection? The ability to clip at an angle is interesting, but this could maybe be done by the selection directly.

@sariug
Copy link
Contributor Author

sariug commented Aug 15, 2022

Its been quite a while since i opened this pr. Thanks for looking at it.

We were using goxel for visualizing outcomes of our operations, and to decide on new operations on some scientific software. in order to implement good ways and angles, it was necessary to be able to have sharp cuts depending on precise normals and precise points.

If i remember correct, the selection tool gives us not enough freedom and precision to have these visualization as quickly as we want.

Also just realized, the code is quite unformatted and probably there exists merge conflicts. I will fix them once you make your mind :)

@guillaumechereau
Copy link
Owner

Nice to see that goxel is used in scientific software! I would be curious to know more about that.

The way I see it I could add support for non aligned box selection, or non align snapping plane (in the 'plane' tool). Maybe both? I think doing it with the plane first is probably the easiest and should allow to do the clipping your branch is doing, without having to modify the layers UI.

It shouldn't be too hard to modify the plane tool to put the same rotation parameters you have in your branch I think, and add a 'clip' button there as well.

I'll check a bit more your commits to see what I can do from it.

@guillaumechereau
Copy link
Owner

A quick test of supporting planes at custom angles seems to work more or less, as shown on this screenshot. There are a a few weird things happening that should be fixed, but I think that would not be too difficult. Would something like that work for your use case?

Screen Shot 2022-08-15 at 7 50 01 PM

@sariug
Copy link
Contributor Author

sariug commented Aug 17, 2022

I played with it a little bit. It seems to be doing the same job.
But it would be really nice to be able to create new layers from different pieces since the model would be pretty big and one would need only hide/show pieces. Think goxel as an investigation tool rather than creation tool. But thanks for the quick modification!

@guillaumechereau
Copy link
Owner

Indeed. I think the best would be to make it work like the fuzzy select tool, where you first make a selection, and then have different possible actions to act on it, including cutting to a new layer. I'll probably change it like that.

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.

None yet

2 participants