Page MenuHomePhabricator

Segmentation view image selectors can be brought to an inconsistent state.
Closed, WontfixPublic

Description

Open an image, create a segmentation. Check the binary property on the image, then uncheck it on the segmentation. As a consequence, both images will be in the segmentation image selector, while none being in the patient image selector.

Event Timeline

The problem is located in the OnBinaryPropertyChanged() method, which is called only upon unchecking the segmentation binary property. The patient image uninentionally goes to the segmentation image selector due to its binary property. Then, the function returns prior to consideration of the segmentation image binary property swap.

User brehler has pushed new remote branch:

bug-18167-InconsistentStateOfImageSelector

[2c4d38]: Merge branch 'bug-18167-InconsistentStateOfImageSelector'

Merged commits:

2015-09-30 14:33:32 Michael Brehler [875f39]
removed isBinary from if condition that avoids the correct addition of the property observer

Reopen: The current solution leads to a crash when a planar figure node is inserted into the DatStorage.

Reproduce:

Reopen: The current solution leads to a crash when a planar figure node is inserted into the DatStorage.

Reproduce:

  • open an image
  • open both SegmentationView and Measurement
  • select measurement view and add some planar figure --> IMMEDIATE CRASH in the QmitkSegmentationView::NodeAdded method

It is because of the removal of the ( isBinary ) check in the if() condition for adding the observer.

[595964]: Merge branch 'bug-18167-InconsistentImageSelectorState-REVERT'

Merged commits:

2015-10-14 16:49:13 Jan Hering [36f1a2]
Revert commit [875f39] merged on 30-09-2015

  • the removed "isBinary" check causes a multiple crashes when handling with non-image objects (planar figures, fibertracts etc.) while the SegmentationView is opened somewhare.

Sorry, but I had to revert the change in QmitkSegmentationView since the initial fix makes the MITK Diffusion application unusable, and also as described above a crash in MITK Workbench with planar figures can be easily provoked.

This bug seems to be unlikely to appear. If possible we should fix it for the upcoming release, however it is not a blocker

User heime has pushed new remote branch:

bug-18167-InconsistentImageSelectorState-Fix

[4604ef]: Merge branch 'bug-18167-InconsistentImageSelectorState-Fix'

Merged commits:

2016-04-13 14:54:04 Eric Heim [288b9c]
Add observers only to images, binary and labelset images

kislinsk added a project: Auto-closed.
kislinsk added a subscriber: kislinsk.

Hi there! 🙂

This task was auto-closed according to our Task Lifecycle Management.
Please follow this link for more information and don't forget that you are encouraged to reasonable re-open tasks to revive them. 🚑

Best wishes,
The MITK devs