BrownBiomechanics / SlicerAutoscoperM

This 3D Slicer extension enables users to perform image registration.
https://autoscoperm.slicer.org
MIT License
1 stars 4 forks source link

ENH: Update Pre-Processing UI to improve BVR workflow #141

Open sbelsk opened 2 weeks ago

sbelsk commented 2 weeks ago

Resolves https://github.com/BrownBiomechanics/SlicerAutoscoperM/issues/131, resolves #145.

These changes are aimed to improve the user workflow in the Pre-Processing module, specifically targeting the improvement of the Autoscoper configuration file generation. Further changes to improve error handling to make it more user-friendly are to be continued in a separate upcoming PR.

Changes in this PR include:

Remaining TODOs:

Existing Pre-Processing UI New Pre-Processing UI
image Screenshot 2024-11-20 133145
amymmorton commented 2 weeks ago

preproc_cfgeduts

sbelsk commented 1 week ago

@amymmorton In the last commit pushed to this branch, I moved a few of the UI widgets so that the pre-processing tab isn't so wide, made sure that the segmentation generation options are enabled/disabled based on which radio button is selected (automatic segmentation vs. seg from models), and also implemented the file selectors for the config generation group.

Current state of the UI: image

amymmorton commented 1 week ago

'or enter:' label to 'AND enter:'

sbelsk commented 1 week ago

Remaining TODOs at this point:

Optional/Future PR:

sbelsk commented 1 week ago

Current status:

amymmorton commented 1 week ago

"populate the "Voxel Size" according to the input volume node selected (should we let users have the ability to manually override it too?)"

I say No. The only options are voxel size form volume node / volume.tfm (which must both be the same)

amymmorton commented 1 week ago

Nice.

image

Suggest

sbelsk commented 4 days ago

Cleaned up the commit history on this branch for ease of reviewing