Page MenuHomePhabricator

[Segmentation] New render window specific reinit warning is confusing
Closed, ResolvedPublic

Description

Here's a test project containing Pic3D and US4DCyl both with segmentations:

When you wanted to segment on an "non-reinited" segmentation, there used to be a simple, immediate warning that you should reinit the segmentation first. It was already hard to understand for users as they probably do not know what "reinit" is and where to find it.

Now the warning is even more cryptic: "Please perform a reinit on the segmentation image inside the entered Render Window!". What is reinit? What is a Render Window? What does entered mean? None of this will make sense for the common user.

On top of that the warning now is only shown when a tool is active and the cursor is on a render window. And there's more cryptic text as overlay: "Interaction is not possible because the render window is not aligned." Again, what is a render window? What does align mean? Aligned to what?

I understand that all these changes were done with the mxn multi widget in mind, but in my opinion it made the situation worse for our common use case of having the std multi widget. Even without all the tech speak the warnings are not only unnecessarily delayed now for this case, but also wrong, since there is no render window specific reinit as far as I know. In addition, I can still interact with the "blocked" render window and continue using tools.

We definitely need to fix and improve the current situation before the upcoming release.

Revisions and Commits

rMITK MITK
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

kislinsk triaged this task as Unbreak Now! priority.Oct 19 2022, 8:53 AM
kislinsk created this task.
kislinsk moved this task from Backlog to Segmentation on the MITK (v2022.10) board.
kislinsk moved this task from Backlog to MITK Meeting on the Request for Discussion board.
kislinsk updated the task description. (Show Details)
kalali added a revision: Restricted Differential Revision.Oct 19 2022, 6:35 PM

There are valid points. We should not confuse the user.
Correctly done, I think the new overlay style is the way to go. Besides optimized/userfriendly text (+making the overlay blocky), we could also directly add icon and btns at the overlay to allow reinit of the render window or all render windows. I think from a UX perspective it would be a huge improvment compared to all other versions.



"Segmentation is currently not possible. This render window is not aligned with the image to be segmented. Please allign this or all render windows to performe the segmentation. (Caution: This may change the current field of view!)"

[Allign] [Allign all]

kalali removed a project: Request for Discussion.

The warning was reverted for the StdMultiWidget. More changes (problems were already mentioned here in this task) will be further implemented using T29335: [Segmentation] Show interactive warning for uninitialized render window geometry.