Skip to content
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

Open
oesteban opened this issue Nov 1, 2021 · 5 comments
Assignees

Comments

@oesteban
Copy link
Member

oesteban commented Nov 1, 2021

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)

@oesteban oesteban self-assigned this Nov 1, 2021
@oesteban
Copy link
Member Author

oesteban commented Nov 2, 2021

@mgxd can you check if either EstimatedTotalReadoutTime or EstimatedEffectiveEchoSpacing are present in the metadata of those Philips scans we've got on Sherlock?

@mgxd
Copy link
Contributor

mgxd commented Nov 2, 2021

No they are not - though if we had access to the DICOMs, we could try converting with the latest dcm2niix

@neurolabusc
Copy link

neurolabusc commented Nov 2, 2021

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.

@manuegrx
Copy link

manuegrx commented Apr 23, 2024

Hi,

This issue is quite old but, unless I missing something, it seems still relevant.
In my lab, we are using fMRIPrep for Philips data, and currently we add the field "TotalReadoutTime" in the json populated by dcm2niix in order to allow fMRIPrep to work for susceptibility distortion correction (using PEPOLAR techniques).

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"

@neurolabusc
Copy link

This is a limitation of the impoverished Philips DICOMs, not dcm2niix. Philips DICOMs are missing information required to determine BIDS tags like TotalReadoutTime. Please lobby the Philips clinical scientist associated your site to provide the required sequence details in the DICOM image files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants