SCIInstitute / ShapeWorks

ShapeWorks
http://sciinstitute.github.io/ShapeWorks/
Other
103 stars 32 forks source link

MoveAlongPCA to use a template mesh in place of mean #211

Open prafulag opened 5 years ago

prafulag commented 5 years ago

It could be useful if this code can work on a template mesh along with its set of correspondence points in place of only the mean mesh.

sheryjoe commented 5 years ago

Added.

prafulag commented 4 years ago

Extend the same to ReconstructSubjectMeshes and also an xml to provide good points file.

prafulag commented 4 years ago

@sheryjoe : This functionality should be further generalized to provide multiple template meshes, so that we do not repeat the transform estimation if multiple templates are being warped. Current usecase: Warping surfaces involved with coverage computation. Please let me know if I should add the way to provide multiple template shapes, if user wants.

sheryjoe commented 4 years ago

What do you mean by multiple templates? The transformation has to be computed if the correspondences (particles) change.

prafulag commented 4 years ago

This template mesh is added to facilitate subject-wise custom surfaces, such as lunate and femoral head for the coverage computation. Often multiple custom subject-specific surfaces are needed from each shape. Currently, to generate multiple surfaces from same template shape, we would run ReconstructSurface multiple times by simply replacing the template mesh name, keeping the points same. I just wanted to cut down on multiple iterations and just one warp estimate is done to warp all the required custom templates to specific subjects. Does that make sense ?

sheryjoe commented 3 years ago

@archanasri Is this command consolidated?

archanasri commented 3 years ago

@sheryjoe Nope, not yet.

archanasri commented 3 years ago

Note: This is Libs/Mesh/ReconstructSamplesAlongPCAModes

archanasri commented 3 years ago

@sheryjoe Can we move this to 6.3? We can focus on consolidation of particle-based executables for the next release

sheryjoe commented 3 years ago

sounds good.

archanasri commented 3 years ago

Relevant to #1018

cchriste commented 2 years ago

Note: This is Libs/Mesh/ReconstructSamplesAlongPCAModes

This executable has been consolidated. Is this issue complete?