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] Incorrect description of DWI "TRACE" image #1862

Open
Lestropie opened this issue Jun 24, 2024 · 4 comments · May be fixed by bids-standard/bids-examples#452 or #1864
Open

[BUG] Incorrect description of DWI "TRACE" image #1862

Lestropie opened this issue Jun 24, 2024 · 4 comments · May be fixed by bids-standard/bids-examples#452 or #1864
Labels
bug Something isn't working diffusion MRI

Comments

@Lestropie
Copy link
Collaborator

Describe your problem in detail.

Prompted by work happening in #1831, but is actually a consequence of #1725.

Having acquired data with all scanner-generated derivatives enabled, I can confirm that the "*_TRACEW" image produced by the Siemens product DWI sequence is not the trace of the diffusion tensor, as the specification currently states; rather, it is a "trace-weighted image", being the mean intensity of b>0 data.

(Unfortunately I did not think to obtain multi-shell data to find out whether the sequence does what one would naturally expect for multi-shell data, being a 4D series with a trace-weighted image volume per unique b-value. It's possible that whether or not it does this will depend on the scanner base software version / sequence used.)

Describe what you expected.

No response

BIDS specification section

https://bids-specification.readthedocs.io/en/latest/modality-specific-files/magnetic-resonance-imaging-data.html#diffusion-imaging-data

@Lestropie Lestropie added bug Something isn't working diffusion MRI labels Jun 24, 2024
@effigies
Copy link
Collaborator

Is the proposal to create TRACEW to mean trace-weighted in addition to, or in lieu of TRACE?

@Lestropie
Copy link
Collaborator Author

I would personally opt to:

  1. Rename TRACE to trace
  2. Change description to reflect that the content of these scanner-generated images is the mean b>0 image intensity, not the trace of the estimated tensor

I'm not aware of any DWI sequence that exports as a scanner-generated derivative the trace of the estimated tensor, so don't see any merit in keeping both parametric maps described alongside one another in the specification. If there's another vendor that does generate such data, happy to stand corrected, I have little to no experience outside of Siemens. But I think this is best treated as an erroneous description.

@effigies
Copy link
Collaborator

cc @neurolabusc I believe the ADC/TRACE definitions were motivated by your use case. Does this make sense to you?

@neurolabusc
Copy link

@effigies on this topic I am a user and not an expert. Following advice from experts, our existing BIDS compatible dataset on OpenNeuro provides images generated by the scanner manufacturer with names like sub-1001_rec-TRACE_dwi.nii.gz and sub-1001_rec-ADC_dwi.nii.gz in the dwi folder, not a derivatives folder. This archive includes data from GE, Siemens and Philips, and it is clear that some variation exists in terms of the formula for ADC and TRACE.

@Lestropie is doing a great job of developing a community consensus on diffusion topics.

Lestropie added a commit to Lestropie/bids-specification that referenced this issue Jun 27, 2024
- Rename "TRACE" suffix to "trace".
- Add "exponentiated ADC" measure.
- Fix description of trace-weighted image, resolving bids-standard#1862.
- Modify descriptions of FA and colFA.
- Fix alphabetic sorting of suffices.
@Lestropie Lestropie linked a pull request Jun 27, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working diffusion MRI
Projects
None yet
3 participants