Skip to content

BUG: Correct warning display bug in GrowCutEffect

Sam Horvath requested to merge github/fork/agirault/fix-growcuteffect-warning into master

Created by: agirault

(1) Corrects wrongfully named method confirmOkCancel to confirmOkCancelDisplay introduced to replace manually created QMessageBox in commit 53f7c53c, and implemented in slicer.util.py.

This was creating the error below when trying to apply the filter with a non-short image:

Traceback (most recent call last):
  File "/Users/alexis/Projects/Dartmouth/nirfastslicer-rel/S-bld/Slicer-build/lib/Slicer-4.5/qt-scripted-modules/EditorLib/GrowCutEffect.py", line 93, in onApply
    if not slicer.util.confirmOkCancel("Current image type is '{0}' and labelmap type is '{1}' GrowCut only works "
AttributeError: 'module' object has no attribute 'confirmOkCancel'

(2) Correct typos in warning message.

Merge request reports