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.

Sign up for GitHub

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

[ENH] Add FA images #1831

Closed
wants to merge 17 commits into from
Closed

[ENH] Add FA images #1831

Show file tree
Hide file tree
Changes from 9 commits
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
e5d9e04
add FA
CPernet May 23, 2024
691e73d
FA suffix
CPernet May 23, 2024
45c14fd
add FA
CPernet May 23, 2024
2e4627e
Update src/schema/rules/files/raw/dwi.yaml
CPernet May 27, 2024
4040942
Update src/modality-specific-files/magnetic-resonance-imaging-data.md
CPernet May 27, 2024
0b3a498
add colFA
CPernet May 27, 2024
f28e539
add colFA
CPernet May 27, 2024
74220cf
colFA description
CPernet May 27, 2024
12b780e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] May 27, 2024
081a59f
fix(schema): Rename raw reference in derivative rules
effigies May 29, 2024
b78b033
Update src/modality-specific-files/magnetic-resonance-imaging-data.md
CPernet Jun 21, 2024
583acd1
Update src/modality-specific-files/magnetic-resonance-imaging-data.md
CPernet Jun 21, 2024
5cabca0
Update src/schema/objects/suffixes.yaml
CPernet Jun 21, 2024
2dd88b6
Merge branch 'master' into add-FA-images
CPernet Jun 28, 2024
ac7584e
all lower case
CPernet Jun 28, 2024
3e991a5
all lower case
CPernet Jun 28, 2024
7c492c2
scanner deriv as lower case.yaml
CPernet Jun 28, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions src/modality-specific-files/magnetic-resonance-imaging-data.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -667,7 +667,7 @@ The definitions of these fields can be found in
and a guide for using macros can be found at
https://github.com/bids-standard/bids-specification/blob/master/macros_doc.md
-->
{{ MACROS___make_suffix_table(["ADC", "TRACE"]) }}
{{ MACROS___make_suffix_table(["ADC", "TRACE", "FA", "colFA"]) }}

<!--
This block generates a filename templates.
Expand Down Expand Up @@ -696,9 +696,9 @@ In such a case, two files could have the following names:
The user is free to choose any other label than `singleband` and
`multiband`, as long as they are consistent across subjects and sessions.

Scanner-generated TRACE and ADC volumes MAY be included using the
`TRACE` and `ADC` suffixes.
If TRACE or ADC volume filenames match a diffusion series with all applicable entities,
Scanner-generated TRACE, ADC and fractional anisotropy (FA and colFA) volumes MAY be included
using the `TRACE`, `ADC`, `FA` and `colFA` suffixes, respectively
CPernet marked this conversation as resolved.
Show resolved Hide resolved
If TRACE, ADC, FA, colFA volume filenames match a diffusion series with all applicable entities,
CPernet marked this conversation as resolved.
Show resolved Hide resolved
such volumes SHOULD be computed from that series.
Otherwise, some entity, such as [`acq-<label>`](../appendices/entities.md#acq),
SHOULD be used to indicate that the files are unrelated.
Expand Down
12 changes: 12 additions & 0 deletions src/schema/objects/suffixes.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,11 +42,23 @@
display_name: Differential interference contrast microscopy
description: |
Differential interference contrast microscopy imaging data
colFA:
value: colFA
display_name: coloured Fractional Anisotropy image

Check failure on line 47 in src/schema/objects/suffixes.yaml

View workflow job for this annotation

GitHub Actions / codespell 

coloured ==> colored
CPernet marked this conversation as resolved.
Show resolved Hide resolved
description: |
Diffusion images reflecting the directionality of the diffusion tensor color-coded in red for
left-right oriented fibers, in blue for superior-inferior oriented fibers and green for
anterior-posterior oriented fibers.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry to chime in here. I may have missed other related discussions, but I'd dare to say that these are usually referred to as DEC (diffusion-encoded colormaps). Most commonly they contain FA information, but could be other information. They are referred to as DECFA in BEP016.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thx for the info! If DECFA is preferred we can just as easily use that, I have no preference but what expert people tell me we should do.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think BEP016 ever uses "DECFA" specifically. The fact that the nature of orientation encoding across volumes is directionally-encoded colour, and that the quantitative parameter encoded in the image is FA, are treated very deliberately as two separate dimensions. The distinction between scanner-generated derivatives and pipeline-generated BIDS Derivatives nevertheless means that there doesn't actually need to be correspondence between the two. Indeed a colour FA image generated by a pipeline will not use this suffix, even though the information encoded is identical.

I maybe have a slight preference for "colFA" just because it's consistent with data being emitted by many scanners that are intended to be encoded in this way.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the explanation.

The distinction between scanner-generated derivatives and pipeline-generated BIDS Derivatives nevertheless means that there doesn't actually need to be correspondence between the two.

Interesting. Did not know about the distinction.

Ad for the label itself, maybe I had a look at the wrong location then: the BEP016 gdocs does mention DECFA when talking about <map_label> fields on page 8: https://docs.google.com/document/d/1cQYBvToU7tUEtWMLMwXUCB_T8gebCotE1OczUpMYW60/edit#heading=h.mqkmyp254xh6
Linked from:
https://bids-specification.readthedocs.io/en/v1.2.1/06-extensions.html

Maybe the google doc should not be linked from the BIDS specification website to avoid confusion?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That BEP016 Google doc is highly outdated; development work on that project transitioned to GitHub some time ago. Maybe that should be flagged near the top of the document, and the website link should be changed. Here's the current status:
bids-standard/bids-bep016#24

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe that should be flagged near the top of the document, and the website link should be changed

Done!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 thanks for doing this.

DF:
value: DF
display_name: Dark-field microscopy
description: |
Dark-field microscopy imaging data
FA:
value: FA
display_name: Fractional Anisotropy image
description: |
Diffusion images reflecting the directionality of the diffusion tensor
FLAIR:
value: FLAIR
display_name: Fluid attenuated inversion recovery image
Expand Down
4 changes: 3 additions & 1 deletion src/schema/rules/files/raw/dwi.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,9 +40,11 @@ sbref:
chunk: optional

# Common scanner-generated derivatives need raw names
isotropic:
ScannerDerivatives:
suffixes:
- ADC
- colFA
- FA
- TRACE
extensions:
- .nii.gz
Expand Down
Loading