-
Notifications
You must be signed in to change notification settings - Fork 65
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
Feat/remove vtk from scalismo #420
Feat/remove vtk from scalismo #420
Conversation
Beside moving all VTK dependent functionality out of scalismo, we also move all active shape model code to a separate repository: https://github.com/unibas-gravis/scalismo-asm |
ASM removed from the code and outcommented tests. I already rebased it to be on top of the release-1.0 branch. So I would say that it can be merged in - it does include the decimator that is still under rewriting and with a bug, bug we have a separate MR open for that. We should just not make a RC before that is fixed and tested. |
* feat: native scala version of Fast-Quadric-Mesh-Simplification --------- Co-authored-by: Andreas Morel-Forster <[email protected]>
Branch rebased to on top of all the other MRs. Comments also removed from tests as requested by @Andreas-Forster . |
Removes all VTK dependent functionality from Scalismo. This is mainly IO and format conversion.
Upstream of this branch needs to be reset to main after the pending MRs are merged in. This branch contains the current version of the MRs.
All vtk dependent tests have been moved to scalismo-vtk and commented out in this MR.
Missing: The
gaussianSmoothing
has been commented out fromGaussianGradientImagePreprocessor
. We need to find another way to implement this in scalismo.By removing the VTK dependency, a simple scalismo.jar app goes from +700MB to ~40MB.
This is the last part of removing the VTK dependency from Scalismo, #415