the default implementation is currently to not overwrite label information if already present in the segmentation image only copy labels from the preview that do not exist.

To change that you would override the TransferPerpare function in your tool class.

in that context one could also think about extending the TransferLabelInformation function, which is used by TransferPrepare and thus the actual copying of label information. Here one could add an option that allows to override label information with the label information of the preview even if it already exists.

Hi Ralf,

Sorry if it wasn't clear, the issue happens already while generating Preview. The TransferPrepare function is only called while confirming labels.
Anyway, the solution (at least partly) lies in modifying the SegWithPreviewTool::TransferLabelInformation. The current logic of checking labels already existing in source / target would need to be changed to achieve what's amicable for MonaiLabel.
When I comment out the check if (!target->ExistLabel(targetLabel, target->GetActiveLayer())), it works. "Spleen" is displayed as spleen in addition to "Label 1".
However, this seems to open more bugs. I have to debug more in this direction.

That ist not a good solution (as far as I can see it) i.a. due to other constraints. Lets discuss this topic in the next meeting.

In the meeting it was decided that a decision regarding whether or not to replace or add labels will be still in RFD.
Decision will be later on after further discussion with the group, in conjunction with Ralf's planned changes in the tool API classes.

Current interim thoughts:

• There is no reason why tools should not be allowed to alter label information at their will
  • also the simple label widget when selecting the labels that should be transfered should show the content of the preview (thus with all alterations the tool made).

• We should allow two modes
  • merge (overwrite the content (label information and pixels) with what we have in the preview (if the preview has changed label name i.a. than it also should change (basically the states of the label (all its content should change; but we cannot just clone and replace the instance, because the other label might be wired with some observers and such...)).
  • add (as new layer) so now colission, but the new labels must also change their labelvalue to avoid collision)
• tool should be able to decide which is its default state (logic is implemented in base class)
• tool UI should be able to decide if the user is allowed to change the mode (logic is implemented in base ui class).

@a178n the changes in D816 should offer everything to solve the problem. I also adapted TotalSegmentator Tool. But you should verify that it works as expected.
Basically the tool should now change the labels in the preview as they like (see Otsu and TotalSegmentator for example) and use the TransferMode AddLabel. Then the labels (with all there settings) of the preview image will be added instead of mapped.

Hi Ralf,

I tested Totalsegmentator. I am seeing inconsistent behavior.

1. the issue at hand seems to be fixed. I can see the label Spleen. But, now the label colours can't be seen anymore. Every label is gray.

2. Colours exist but no labels in the preview list.

Happens randomly.