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

Missing metadata for res and den outputs #752

Open
rwblair opened this issue Oct 13, 2022 · 1 comment · Fixed by #754
Open

Missing metadata for res and den outputs #752

rwblair opened this issue Oct 13, 2022 · 1 comment · Fixed by #754

Comments

@rwblair
Copy link
Contributor

rwblair commented Oct 13, 2022

Derivatives specification says if the res entitiy is used Resolution must be defined in its json file:
https://bids-specification.readthedocs.io/en/stable/05-derivatives/03-imaging.html#preprocessed-coregistered-andor-resampled-volumes

The actual content of the Resolution field is pretty wide open so boiler text explaining the number directly corresponds to millimeter resolution should be good enough.

This is an issue for both anatomical and functional outputs.

@mgxd mgxd reopened this Oct 27, 2022
@mgxd mgxd changed the title Missing metadata for outputs that use the res entitiy. Missing metadata for res and den outputs Oct 27, 2022
@mgxd
Copy link
Contributor

mgxd commented Oct 27, 2022

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

Successfully merging a pull request may close this issue.

2 participants