Page MenuHomePhabricator

[Segmentation][MultiLabel][Utilities] UI is inconsistent / unstable
Open, HighPublic

Assigned To
Authored By
kalali
Oct 29 2021, 3:15 PM
Referenced Files
F2461820: image.png
Nov 2 2021, 3:13 PM
F2461800: image.png
Nov 2 2021, 3:13 PM
F2461795: image.png
Nov 2 2021, 3:13 PM
F2461810: image.png
Nov 2 2021, 3:13 PM
F2461808: image.png
Nov 2 2021, 3:13 PM
F2461814: image.png
Nov 2 2021, 3:13 PM
F2461816: image.png
Nov 2 2021, 3:13 PM
F2461822: image.png
Nov 2 2021, 3:13 PM

Description

Many issues were reported / encountered, related to the UI design of the segmentation utilities.
Additionally many things were found were the UI of both utilities differs.

Segmentation

image.png (165×977 px, 12 KB)

No error here.
image.png (206×1 px, 18 KB)

It clearly is visible. Think this is an issue of comprehending the checklist. Most other testers didn't report this.
image.png (228×1 px, 20 KB)

This relates to T28134.
@c306h

image.png (169×991 px, 15 KB)

Relates to T27810, T28134.
image.png (110×999 px, 11 KB)

Relates to T27810, T28134.
image.png (170×999 px, 15 KB)

Doesn't seem to be an error.

@kalali

MultiLabel Segmentation

image.png (283×938 px, 23 KB)

This relates to T28134.
image.png (151×994 px, 13 KB)

Relates to T28134.

@kausch

image.png (92×726 px, 6 KB)

This is a helpful suggestion to improve the checklist, not a bug in and of itself.
image.png (111×801 px, 7 KB)

This is also true, but its a design feature not a bug. Labels can easily be converted to boolean masks by right clicking on the label and picking the "Convert to Mask" option.
image.png (148×838 px, 10 KB)

This item in the checklist works as expected, but the colors of the masks are not ideal for people unfamiliar with this portion of MITK.
@s349i

image.png (291×1 px, 27 KB)

This relates to T28134.
image.png (142×1 px, 11 KB)

This relates to T28134.
image.png (265×1 px, 18 KB)

This relates to T28134.
image.png (189×995 px, 15 KB)

This probably needs a new error message.
image.png (315×994 px, 19 KB)

This relates to T28134.
image.png (172×1 px, 15 KB)

This relates to T28134.
image.png (123×1 px, 12 KB)

This relates to T27913.
image.png (196×1 px, 16 KB)

This relates to T28134.
image.png (149×999 px, 12 KB)

Boolean operations done in a different timestep persist when the timestep is changed.
image.png (148×999 px, 13 KB)

For boolean operations, the result of union is not a 4d segmentation. It's a 3d segmentation. And If the boolean operation is performed at timestep t=1, then the time step is changed to t=0, the operation result from t=1 is still visible.
image.png (174×999 px, 15 KB)

This relates to T28134.
@kalali

Additionally the following tasks were already opened and are related:

As a result we already decided to remove the redundancy. The following tasks are related to that:

Event Timeline

kalali triaged this task as Normal priority.Oct 29 2021, 3:15 PM
kalali created this task.
kalali renamed this task from [Segmentation][MultiLabel][Utilities] UI is to [Segmentation][MultiLabel][Utilities] UI is inconsistent / unstable.Oct 29 2021, 3:26 PM
kalali moved this task from Backlog to Segmentation on the MITK (v2022.04) board.
floca raised the priority of this task from Normal to High.Sep 7 2022, 5:09 PM
floca moved this task from Tests/Checklists to Segmentation on the MITK (v2022.10) board.
floca added a subscriber: floca.

@s349i Please check which

  • of the problem are already solved by your changes
  • could be easily changed for that release
  • would be still open and should be done in the next release.

@s349i Are those checked now? If not please do this ASAP and verify. Thanks.