-
Notifications
You must be signed in to change notification settings - Fork 27
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
Read Philips' EstimatedTotalReadoutTime
and EstimatedEffectiveEchoSpacing
#249
Comments
@mgxd can you check if either |
No they are not - though if we had access to the DICOMs, we could try converting with the latest dcm2niix |
Philips DICOM images do not provide the required information. This is a limitation of Philips DICOM images, not dcm2niix. For details, see my comments here. |
Hi, This issue is quite old but, unless I missing something, it seems still relevant. I was wondering if it was planned for fMRIPrep (and there for for sdcflows) to read the field "EstimatedTotalReadoutTime" when "TotalReadoutTime" is not available ? Currently, it seems that fMRIPrep is not working without the field "TotalReadoutTime" because the bids validator returns an error (an issue is open on this subject : https://github.com/bids-standard/bids-validator/issues/1588) but, even without this error I am not sure that sdcflows will be able to read the field "EstimatedTotalReadoutTime" |
This is a limitation of the impoverished Philips DICOMs, not dcm2niix. Philips DICOMs are missing information required to determine BIDS tags like |
Recent versions of dcm2niix have adopted these two for Philips scanners, we should check them before we resort to the water-fat shift formula.
See: rordenlab/dcm2niix#389 (
rordenlab/dcm2niix@ab26453
)The text was updated successfully, but these errors were encountered: