Switch new dicom node naming to patientID-Modality
Description
Description
Details
Details
Event Timeline
/Plugins/org.mitk.gui.qt.dicom/src/internal/DicomEventHandler.cpp | ||
---|---|---|
350–351 | I would propose to use the DICOMTagPath and PropertyKey generation helpers instead of "magic number" strings! |
/Plugins/org.mitk.gui.qt.dicom/src/internal/DicomEventHandler.cpp | ||
---|---|---|
333–351 | I am a bit sceptical to change naming this way. This will also often bring name collisions. Every MRI of a single patient will be named equaly!!! I definetly would keep at least the series desciption or use all 4. I think at the end (but not as a hard requirement) it would be best, that we allow the configuration of the node naming patterns. E.g. a mircoservice were you can register for different data types which name template and property keys should be used to generate a concrete name. |
Comment Actions
As an addition to my last review comment. I just stumpled upon a task that already described what I meant (totally forgot about it).
T19477: Select which dicom tags are used to create a nodes name