-
Notifications
You must be signed in to change notification settings - Fork 298
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
KeyError: "Metadata term 'RepetitionTime' unavailable for file #2538
Comments
I also get this error, particularly when working with large pybids database layouts (so large that BIDS validator has issues). |
I guess this would be a PyBIDS bug. Did previous versions of fMRIPrep work on these data? |
I received similar errors when using 20.2.3 and 21.0.0rc0. I have not tried any older version. |
If I remember correctly 20.2.1 worked on these data. Should have mentioned that. |
It would be good if you could retry with those versions so we could try to narrow down the cause.
I notice in the original report:
If you're using pre-computed PyBIDS database files, you may want to delete and re-compute them. I believe there was a change to the database format, and it could have subtle effects rather than throwing an intelligible error. |
I tried reverting to a 0.12.3 PyBIDS version layout on the unstable 21.0.0 release, same error. Have not tried going back to 20.2.1 yet. When having fMRIPrep recompute the dataset I appear to get farther, reaching the numpy import error referenced before (#2532), but obviously for large datasets that's not ideal. |
We definitely don't want to go back to an older version of PyBIDS to create the layout. Apologies if that was unclear. You should use the same version of PyBIDS as fMRIPrep to create the layout. My concern was that you might be reusing a layout created by an older version. |
When I use a premade pybids layout v0.13.1 on 20.2.3 or 0.13.2 on 21.0.0rc0 I get the error in this issue. However, when I have fMRIPrep make the layouts I do not. |
Can you share your command for pre-computing the layout? |
|
Okay, I reproduced this issue, and the fix is that you need to add
It is unclear why that's disabled by default. We should add it to our docs. |
This seems to be resolved. |
What version of fMRIPrep are you using?
21.0.0rc0
What kind of installation are you using? Containers (Singularity, Docker), or "bare-metal"?
Singularity container
What is the exact command-line you used?
Have you checked that your inputs are BIDS valid?
Yes, they are BIDS valid
Are you reusing previously computed results (e.g., FreeSurfer, Anatomical derivatives, work directory of previous run)?
No
fMRIPrep log
If you have access to the output logged by fMRIPrep, please make sure to attach it as a text file to this issue.
#################################################################
Because it's rather short I'll put it in the post directly
The issue is the corresponding json sidecar actually contains the repetition time.
The text was updated successfully, but these errors were encountered: