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

Ensure per-BOLD SyN-SDC does not require TotalReadoutTime #3009

Open
effigies opened this issue May 17, 2023 · 1 comment · May be fixed by #3423
Open

Ensure per-BOLD SyN-SDC does not require TotalReadoutTime #3009

effigies opened this issue May 17, 2023 · 1 comment · May be fixed by #3423
Milestone

Comments

@effigies
Copy link
Member

What would you like to see added in fMRIPrep?

fMRIPrep 23.1.0 will not lump BOLD images together for running SyN-SDC. This should eliminate the need for a dataset-defined TotalReadoutTime, and we can impute something in order to ensure that a plausible fieldmap. 1/16 (0.0625) or 1/32 (0.03125) seconds are reasonable, and being powers of two should mostly avoid rounding errors.

Do you have any interest in helping implement the feature?

Yes

Additional information / screenshots

No response

@effigies effigies added this to the 23.1.0 milestone May 17, 2023
@effigies effigies modified the milestones: 23.1.0, 23.2.0 Aug 24, 2023
@effigies
Copy link
Member Author

This needs to be a CLI flag, as anything else is asking for a compatibility error if fieldmaps are generated in one version and applied in another. Should probably save the imputed TRT as metadata on the fieldmap as well.

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