QuantitativeReporting icon indicating copy to clipboard operation
QuantitativeReporting copied to clipboard

Improve naming of the loaded label nodes

Open fedorov opened this issue 10 years ago • 4 comments

from Steve:

Yes, the convention is SeriesNumber: SeriesDescription (like "3: Ax T1"). Might make sense to do this for all DICOM plugins as a default so maybe some of this code should be a helper method in the DICOMPlugin superclass.

https://github.com/Slicer/Slicer/blob/master/Modules/Scripted/DICOMPlugins/DICOMScalarVolumePlugin.py#L66-L80

fedorov avatar Sep 27 '15 18:09 fedorov

A related effort that was essentially "killed" that should be considered if and when we get to this issue: https://github.com/Slicer/Slicer/pull/527

fedorov avatar Nov 16 '16 02:11 fedorov

@fedorov Maybe this could also be interesting for the project week?

che85 avatar Jan 03 '18 20:01 che85

Yes, maybe. I am not sure though if the default series number + description will work in all cases. Depending on the user expectations, it might help to include the loadable type in the name. I am not sure there is a universal solution.

fedorov avatar Jan 03 '18 20:01 fedorov

We may update MRML node combo box to display patient/study/series information.

Also, @cpinter has implemented a new node selector (qMRMLSubjectHierarchyComboBox) that allows you to select a node from the subject hierarchy tree. After some further testing and tuning we can replace qMRMLNodeComboBox with this (we can keep the same interface, so nothing else would not need to be changed).

lassoan avatar Jan 03 '18 21:01 lassoan