aces / Loris-MRI

The set of scripts that preprocess and insert MRI data into the database.
10 stars 50 forks source link

[dicom tar] Seemingly wrong handling of unknown protocol #1116

Open MaximeMulder opened 2 months ago

MaximeMulder commented 2 months ago

In the Raisinbread, file ImagingUpload-18-41-C4ZlTl.tar, the series localizer is debscribed to have only one associated file. However, there seem to be actually three files associated with this sequence.

https://demo.loris.ca/dicom_archive/viewDetails/?tarchiveID=62

The results are visible in the Raisinbread database, and reproductible by re-taring or re-inserting the archive in the database.

Also, the series is marked to described to have a "COL" phase encoding, when the three corresponding files have a phase encoding of "ROW", "ROW" and "COL".

I do not know DICOMs good enough yet to know the solution here. Should there be three separated series ? Should the phase encoding be NULL ? Opinions welcome here.