Skip to content

BUG: update management of crop volume parameters node

Nicole Aucoin requested to merge github/fork/naucoin/3947-Repeated-crop-volume-freezes-Slicer into master

The module was holding a pointer to the parameter node that could get out of date on scene changes and cause crashes. Switch to using a mrml node combo box to manage the parameters node. Adjusted the UI file to allow the generic module widget test to pass. Update the widget more fully on enter and scene changes. Select a newly added input volume or ROI if none already selected. When picking a new input volume, or switching ot a new parameter set one, set it to be active in the slice viewers. Update the parameters node constructor, destructor, read/write xml, copy and print for missing calls. Make sure that the GUI updates the parameters node (was missing the input volume and ROI node IDs). Added a scene close to the self test, and add the CMake command to make it show up as a self test in the GUI. Added error checking, took out debugging print outs.

Tested against issues: #3947 #1764 #3313

Issue #3947

Merge request reports