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

Added fidcsi and fidall options for GE SVS - for special case of pulse-acquire scans #157

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DC-3T
Copy link
Contributor

@DC-3T DC-3T commented Dec 10, 2024

Added an extra function to allow GE p-files with 'fidcsi' and 'fidall' psds to be processed as SVS scans - in the special case of pulse-acquire data. Previously, the code would treat these pulse-acquire data as SVS data (no phase encoding, spatial dims = [1, 1, 1]) but the corresponding psd names were not included in the list of SVS psd's.

@wtclarke
Copy link
Owner

Hi @DC-3T , Thank you for the pull request. My understanding is that the fidall sequence can be literally anything (completely arbitrary mag-prep and readout), which is cool from a development point-of-view, but a nightmare for handling the data. Is there a way of reliably identifying this case where it is pulse-acquire, from the headers?

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

Successfully merging this pull request may close these issues.

2 participants