We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Instead of:
https://github.com/poldracklab/fmriprep/blob/4c54ca4f855d4f560c7c4abdbbbcb4fb3a60bae6/fmriprep/workflows/bold/base.py#L147
we may want to do something like:
entities = layout.parse_file_entities(ensure_list(ref_file)[0]) multiecho = "echo" in entities
May I suggest to address this in the context of #1803 ?
The text was updated successfully, but these errors were encountered:
Sorry for the delay on responding to this. Is echo prohibited for single-echo data? The specification is unclear:
echo
Multi-echo data MUST be split into one file per echo. Each file shares the same name with the exception of the _echo-<index> key/value.
_echo-<index>
If we assume that single-echo data cannot include echo, then I think the proposed check is good.
Sorry, something went wrong.
Perhaps this is easier than I thought in the beginning. What about testing:
multiecho = len(entities.pop("echo", [])) > 2
I believe that covers one of the edge cases of #2182 too.
enh: revise multiecho identification, sync with nipreps/niworkflows#408
31d646b
Resolves: nipreps#2179.
tsalo
Successfully merging a pull request may close this issue.
Instead of:
https://github.com/poldracklab/fmriprep/blob/4c54ca4f855d4f560c7c4abdbbbcb4fb3a60bae6/fmriprep/workflows/bold/base.py#L147
we may want to do something like:
May I suggest to address this in the context of #1803 ?
The text was updated successfully, but these errors were encountered: