-
Notifications
You must be signed in to change notification settings - Fork 228
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
Possibility to ask for a specific manufacturer tag to split output nifti #384
Comments
2 This sounds like the expected behavior. Does |
I just sent you some data. |
@chrisgorgo historically you have been the maintainer for BIDS compatibility from dcm2niix. Do you want to continue this role? Alternatively, now that you have left the ivory tower do you want to pass the role to someone else (e.g. @effigies)? We always have a choice whether 4D series are saved as a single NIfTI (e.g. DWI, fMRI) or split into separate files (e.g. multi-echo sequences). Can you provide guidance for how ihMT data should be handled? I have no preference, I just want my handling to be consistent with the standard. Users can always employ fslsplit or fslmerge if they are unhappy with the result. |
Hi I'm only guessing - but I suspect this will be a research sequence without any associated descriptive dicom tags and will probably use the dyanamic axis (4D sets can have have several different 4th dimensions described) with the dynamic looping re-purposed for the MT changes. M |
@arnaudbore thanks for a sample. I note that different volumes have unique trigger times (0018,1060). I think this can be handled with the _t filename post-fix, which is also used for Trigger delay time (0020,9153). GE seems to use this tag differently, but I think my patch avoids any unintended consequences. Why don't you try out the developmental branch and see what you think. As @drmclem notes, you are using a WIP (Work In Progress) sequence, so it is often worth waiting until these protocols graduate to product sequences before modifying a general purpose tool like dcm2niix. |
This particular case may be covered already by BEP 001. I will tag in @Gilles86, as he will certainly be more familiar with both the scan sequences and the proposed solutions. |
@neurolabusc, the development branch seems to do exactly what I was looking for. |
* tag 'v1.0.20200331': (52 commits) Update submodules Update dcm_qa submodule. UIH 3D sequence quirk New release, EstimatedTotalReadoutTime/EstimatedEffectiveEchoSpacing (rordenlab#377) Philips TotalReadoutTime (rordenlab#377) Cleanup Experimental Canon DICOM support (rordenlab#388) Experimental solution for issue 384 (rordenlab#384) Detect catastrophic anonymization (rordenlab#383) Only report "multiple inversion times" if 0018,9079 values differ (e.g. Bangalore data in https://github.com/neurolabusc/dcm_qa_philips) Consistent echo naming (rordenlab#381) Philips partial Fourier (rordenlab#377) Support InversionTImes (0018,9079) tag (rordenlab#380) Philips effective echo spacing formula ambiguous (rordenlab#377) TR for Philips 3D EPI (rordenlab#369) Citation (rordenlab#102) GE PET with variable slice intensity (rordenlab#374) Estimate Philips EffectiveEchoSpacing (nipreps/sdcflows#5) GE slice interpolation (rordenlab#373) 3D EPI TR (rordenlab#369) 3D phase (rordenlab#371) Enhanced ordering (rordenlab#372 (comment)) ...
Hi,
I've got some Philips dicom (inhomogeneous Magnetization Transfer images) I need to split into folders before running dcm2niix otherwise it will combine them together as one image. I think it could be fixed if I was able to split the output nii.gz file with a specific tag into the dcm2niix command line. I'm looking at a tag known as TemporalPositionIdentifier. Also to run my scripts I would need these tags as well [0x2005, 0x0100d] and [0x2005, 0x0100e]. It would be nice to have a way to extract these values into the json file.
Thank you in advance
Arnaud
The text was updated successfully, but these errors were encountered: