Required modifiers in terminology

We would like to use terminologies correctly in NVidia’s AI segmentation plugin. However, the terminology json or logic does not seem to make sense. For example, I cannot create a segment for “spleen”. When I choose “Segmentation category and type – DICOM master list” and search for spleen, a property is found, but I’m forced to select laterality! I can choose “Left”, “Right”, or “Right and left” - but there is no way I can simply choose “Spleen”.

image

Is this an error in the terminology json (there should be an option for not specifying laterality)? Or is this an error in the terminology implementation (it should not require selection of a modifier)?

https://github.com/Slicer/Slicer/blob/ef1891672d53b8dba745673f1ca6eafd1fbfc473/Modules/Loadable/Terminologies/Resources/SegmentationCategoryTypeModifier-DICOM-Master.json#L12281-L12307

@fedorov @cpinter

The behaviour is expected, this is how I implemented it back then, based on the specification and later clarifications of @fedorov.

If it needs to be changed, then it is not an implementation question but one related to the standards.

1 Like

@fedorov, can you explain the rationale for requiring selection of “Left”, “Right”, or “Right and left” for organs such as the spleen?

Spleen is defined in the dcmqi segmentation JSON file, which (at least, a version of it) I believe was used to populate segment editor, here:

It is marked as unpaired, so I do not know why laterality is forced in Segment editor.

I think the “paired” attribute was not included in the specifications when implementing Terminologies, thus it is not considered.

Spleen has laterality in this file in the dmqi repository:

Which file Slicer should use?

I think this discussion from March 2017 should help: https://github.com/QIICR/QuantitativeReporting/issues/150#issuecomment-288408153

It might well be that this was not considered at the time. The reality is I don’t know how frequently this feature in Slicer was used, so chances are things went un-noticed.