View Issue Details

IDProjectCategoryView StatusLast Update
0003937Slicer4Module DICOMpublic2018-05-29 23:26
Reporterpinter Assigned Topinter  
PrioritynormalSeverityfeatureReproducibilityN/A
Status assignedResolutionopen 
Product VersionSlicer 4.4.0 
Target VersionbacklogFixed in Version 
Summary0003937: Add exported DICOM series to existing study
Description

In the current state of DICOM export it seems that it is not possible to add the exported DICOM series under an existing study, as a new one is created each time.

Both StudyInstanceUID and StudyID need to match in order for the database to recognize it as the same study.

TagsNo tags attached.

Activities

pieper

pieper

2015-01-09 09:29

administrator   ~0012875

Csaba and I talked about this. He will look at this and report more information.

I would say that if the studyUID is different than it should be considered a different study.

pinter

pinter

2015-01-09 09:34

developer   ~0012876

I basically need to investigate why the StudyInstanceUID of the exported series does not match the original study UID and fix it.

pinter

pinter

2015-01-27 12:48

developer   ~0012889

Right now the plugins are responsible for creating study instance UIDs, and they will need to support adding exported series to existing study.

See conversation
http://slicer-devel.65872.n3.nabble.com/Create-Dicom-Series-possoble-improvement-td4033353.html

pinter

pinter

2015-01-27 12:53

developer   ~0012890

How I imagine supporting this on the DICOM export level: the study nodes need to store their UIDs if imported from DICOM, and if it exists, then the exporter only exports the new series under that, passing the study UID to the plugins.

pinter

pinter

2016-06-06 12:20

developer   ~0013947

Fixed in the RT export plugin, see further information
https://www.assembla.com/spaces/slicerrt/tickets/668

Need to see how it works for scalar volume export.

pinter

pinter

2017-03-29 16:03

developer   ~0014391

Scalar volume exporter seems to generate new study instance UID every time.

There is no trace of re-using study instance UID neither in DICOMExportScalarVolume.py nor in CreateDICOMSeries.cxx. It seems that itk::ImageFileWriter<Image2DType> with itk::GDCMImageIO generates a new one each time. Also the Study ID (which is just a string) is set to a fixed SLICER10001, so apparently by design it is not possible to append to existing study.

pieper

pieper

2017-03-29 16:29

administrator   ~0014392

We should really consider re-writing the CreateDICOMSeries.cxx functionality. It was put together by the team at GE long ago with the very restricted goal of getting a particular GE system to read the data. It would not be hard to meet or exceed those goals with a little work and we could avoid needing to work around existing limitations.

pinter

pinter

2017-03-29 16:31

developer   ~0014393

That was my conclusion as well. However I don't have the capacity to work on this in the foreseeable future unfortunately.

pieper

pieper

2017-03-29 16:33

administrator   ~0014394

Understood. We may end up with some building blocks from the qiicr / dcmqi project or similar that will make this easier.

lassoan

lassoan

2018-05-29 23:26

developer   ~0015783

Retargeting stale issues to backlog.

Issue History

Date Modified Username Field Change
2015-01-08 10:56 pinter New Issue
2015-01-08 10:56 pinter Status new => assigned
2015-01-08 10:56 pinter Assigned To => pieper
2015-01-09 09:29 pieper Note Added: 0012875
2015-01-09 09:29 pieper Status assigned => feedback
2015-01-09 09:34 pinter Status feedback => assigned
2015-01-09 09:34 pinter Assigned To pieper => pinter
2015-01-09 09:34 pinter Note Added: 0012876
2015-01-27 12:48 pinter Note Added: 0012889
2015-01-27 12:53 pinter Note Added: 0012890
2016-06-06 12:20 pinter Note Added: 0013947
2017-03-29 16:03 pinter Note Added: 0014391
2017-03-29 16:29 pieper Note Added: 0014392
2017-03-29 16:31 pinter Note Added: 0014393
2017-03-29 16:33 pieper Note Added: 0014394
2018-05-29 23:26 lassoan Target Version => backlog
2018-05-29 23:26 lassoan Note Added: 0015783