-
Notifications
You must be signed in to change notification settings - Fork 229
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Multi-echo naming problem with Vida fit classic DICOMs #568
Comments
This is related to issue 538. The XA30 data format has evolved rapidly (in a good way), but I have very little access to sample datasets. Feel free to send a sample dataset to me (compress the DICOMs, share a link to the compressed file with the personal email in my avatar). My other suggestion would be to export the data as enhanced DICOM instead of classic DICOM. See issue 236 for more details. |
Thanks for the informative response - I have sent you an email. :) |
As you note, these DICOMs omit echo number (0018,0086). This is a type 3 attribute, so these are technically valid DICOMs. However, I believe this was resolved with XA30 (and perhaps on your XA20 is you export image as enhanced rather than DICOM). I believe that the XA20 was a stop gap between the impoverished XA10/XA11 images and the more mature XA30. I would certainly see if this instrument is eligible for an upgrade. I worry about the maintainability of adding code to handle this transient release (indeed, the code is already littered with kludges for the early XA aberrations). Please try out the commit to the development branch (v1.0.20211220) which explicitly inserts the echo time into the filename for multi echo sequences where 0018,0086 is omitted. Alternatively, you can use the Matlab based dicm2nii which includes the logic to infer the echo number. As an aside, I do think that the echo number is reported as the first field in the proprietary ICE dims (0021,1106). For example, the 99th slice of the 4th echo reports
|
Describe the bug
Multi-echo data acquired using Siemens' Vida fit, and exported as classic DICOM are not named properly by dcm2niix in relation to echo numbers when the
Echo Number
field of the DICOM header is not present. The resultant names produced by dcm2niix indicate that each echo was interpreted to be a single-echo scan from a different 'run', rather than multiple echoes from the same run. It should be possible to fix this by inferring the echo numbers via the required DICOMEcho Time
field rather than relying on the presence of the optionalEcho Number
field.To reproduce
Steps to reproduce the behavior:
Run the command
dcm2niix .
in a folder containing multi-echo Siemens Vida fit data exported as classic DICOM with noEcho Number
field.See the output below. Take note of the filenames and echo times in the final output:
Expected behavior
Each file should be named according to the inferred echo number based on the temporal order of the required
Echo Time
field, rather than the optionalEcho Number
field. For instance:Version
Troubleshooting
The text was updated successfully, but these errors were encountered: