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

Selection tool feature #46

Closed
wants to merge 12 commits into from

Conversation

mauigna06
Copy link
Contributor

This branchs adds the Dynamic Modeler tool 'Selection'. It takes a fiducialNode and a modelNode as input. It selects points of the modelNode around the distanceParameter with two possible algorithms (insideSphereRadius or geodesicDistance). The selected output can be the selected faces or the input model with selection scalars (1s and 0s).

dynamicModelerSelectionTool

This work will impact all modelToModel registrations made with Slicer allowing it to be as competent in this area as other software such as Blender.

@mauigna06
Copy link
Contributor Author

I'm not sure what to write instead of the original copyright string

@mauigna06
Copy link
Contributor Author

mauigna06 commented Oct 28, 2021

Also: there must be a way to set up the locator only once (unless the inputMesh changes) and maybe save some computing time.
I also could make the RunInternal function call a timer that has a callback 100ms later when there is no more user-interaction. Would this be useful?

@lassoan
Copy link
Contributor

lassoan commented Oct 29, 2021

This is awesome, thanks a lot, I'll test this and give feedback soon.

@lassoan
Copy link
Contributor

lassoan commented Oct 31, 2021

Thank you, I've tested this and it worked amazingly well. I've simplified the code the bit, added caching of a few filters that takes a long time to reinitialize to improve performance for complex models, etc. squashed all the commits and merged in this commit: 3e42453

@lassoan lassoan closed this Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants