Page MenuHomePhabricator

[Selection concept] Single node selection widget of ML view reacts if ML view is hidden
Closed, InvalidPublic

Description

We found out while working on https://phabricator.mitk.org/D523#16388 that the MultiLabel plugin view reacts on a new empty segmentation that has been added to the data storage by another plugin (e.g. the segmentation plugin view). The auto selection mode leads to a call to QmitkSingleNodeSelectionWidget::OnNodeAddedToStorage which in turn triggers QmitkMultiLabelSegmentationView::OnSegmentationSelectionChanged. We noticed this because the call to QmitkMultiLabelSegmentationView::OnSegmentationSelectionChanged initializes the views which was undesired.

The general problem here is that the QT signal-slot connection to QmitkAbstractNodeSelectionWidget::CurrentSelectionChanged is still valid, even if the MultiLabel plugin view is hidden.
This might be a general problem for different selection-changes and will not only occur with the two mentioned plugin views.

Event Timeline

kalali triaged this task as High priority.Aug 9 2021, 2:03 PM
kalali created this task.
kalali added a project: Request for Discussion.
kalali lowered the priority of this task from High to Normal.Aug 11 2021, 11:42 AM
kalali moved this task from MITK Meeting to Backlog on the Request for Discussion board.
kalali edited projects, added MITK; removed MITK (v2022.04).

The QmitkMultiLabelSegmentationView has been removed with D601, so I will close this task.
However, as mentioned, this is a general problem with views that use a node selection widget. So we might re-open this task if this problem occurs again in a different scenario.