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

[BUG] fsLR template description is (probably) inaccurate #1901

Open
tsalo opened this issue Aug 21, 2024 · 1 comment · May be fixed by #1927
Open

[BUG] fsLR template description is (probably) inaccurate #1901

tsalo opened this issue Aug 21, 2024 · 1 comment · May be fixed by #1927
Labels
bug Something isn't working derivatives MRI For things that affect all MRI datatypes

Comments

@tsalo
Copy link
Member

tsalo commented Aug 21, 2024

Describe your problem in detail.

fsLR:
value: fsLR
display_name: fsLR
description: |
The `fsLR` is a **dual template** providing both volumetric and surface coordinates references.
The volumetric template corresponds to `MNI152NLin6Asym`.
Surface templates are given at several sampling densities:
164k (used by HCP pipelines for 3T and 7T anatomical analysis),
59k (used by HCP pipelines for 7T MRI bold and DWI analysis),
32k (used by HCP pipelines for 3T MRI bold and DWI analysis), or
4k (used by HCP pipelines for MEG analysis) fsaverage_LR surface reconstructed from the T1w image.
Used by Freesurfer.

The fsLR template isn't actually a dual template. The combination of the fsLR surface with MNI152NLin6Asym volume is a convention, not a rule.

Describe what you expected.

fsLR should only describe the surface template. See #1900 for discussion of the distinction between the surface and volume templates in combined surface-volume files (i.e., CIFTIs).

BIDS specification section

https://bids-specification.readthedocs.io/en/stable/glossary.html#fslr-enums

https://bids-specification.readthedocs.io/en/stable/appendices/coordinate-systems.html#standard-template-identifiers

@tsalo tsalo added bug Something isn't working derivatives MRI For things that affect all MRI datatypes labels Aug 21, 2024
@tsalo
Copy link
Member Author

tsalo commented Aug 22, 2024

From maintainers meeting today- folks are comfortable using fsLR as shorthand for fsLR+MNI152NLin6Asym. However, I still think we need to clarify that in the documentation, since a CIFTI with fsLR surfaces and some other volumetric space is still in fsLR (though one would need to differentiate it from standard fsLR), and that might not be clear from the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working derivatives MRI For things that affect all MRI datatypes
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant