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

Jump to bottom

[ENH] Add generic metadata from BEP22 to MRI #1396

Merged
merged 9 commits into from
Apr 11, 2024
Merged

[ENH] Add generic metadata from BEP22 to MRI #1396

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
42925e0
ENH: Add sample metadata to MRI and PET
Remi-Gau Feb 5, 2023
3664403
ENH: Add generic metadata from BEP22 to MRI
Remi-Gau Feb 5, 2023
fb1f5cc
Merge remote-tracking branch 'upstream/master' into extra_mri_metadata
Remi-Gau Aug 24, 2023
22073a5
rm sample
Remi-Gau Aug 24, 2023
a9126e4
Update src/schema/objects/metadata.yaml
markmikkelsen Apr 10, 2024
f5f554b
Update src/schema/objects/metadata.yaml
markmikkelsen Apr 10, 2024
bfc116d
Update src/schema/objects/metadata.yaml
markmikkelsen Apr 10, 2024
f934160
rm macros with missing metadata
Remi-Gau Apr 11, 2024
f3dd861
Merge remote-tracking branch 'upstream/master' into extra_mri_metadata
Remi-Gau Apr 11, 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
38 changes: 38 additions & 0 deletions src/schema/objects/metadata.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -232,6 +232,20 @@ B0FieldSource:
- type: array
items:
type: string
B0ShimmingTechnique:
name: B0ShimmingTechnique
display_name: B0 Shimming Technique
description: |
The technique used to shim the *B<sub>0</sub>* field (for example, `"Dynamic shim updating"`
or `"FASTMAP"`).
type: string
B1ShimmingTechnique:
name: B1ShimmingTechnique
display_name: B1 Shimming Technique
description: |
The technique used to shim the *B<sub>1</sub>* field (for example, `"Simple phase align"`
or `"Pre-saturated TurboFLASH"`).
type: string
BIDSVersion:
name: BIDSVersion
display_name: BIDS Version
Expand Down Expand Up @@ -2214,6 +2228,12 @@ NumberOfVolumesDiscardedByUser:
`"NumberOfVolumesDiscardedByUser"` field.
type: integer
minimum: 0
NumberReceiveCoilActiveElements:
name: NumberReceiveCoilActiveElements
display_name: Number of Receive Coil Active Elements
description: |
The number of active RF elements used by the receive coil.
type: integer
NumberShots:
name: NumberShots
display_name: Number Shots
Expand All @@ -2233,6 +2253,12 @@ NumberShots:
- type: array
items:
type: number
NumberTransmitCoilActiveElements:
name: NumberTransmitCoilActiveElements
display_name: Number of Transmit Coil Active Elements
description: |
The number of active RF elements used by the transmit coil.
type: integer
NumericalAperture:
name: NumericalAperture
display_name: Numerical Aperture
Expand Down Expand Up @@ -3594,6 +3620,18 @@ VolumeTiming:
items:
type: number
unit: s
WaterSuppression:
name: WaterSuppression
display_name: Water Suppression
description: |
Boolean indicating whether water suppression was used prior to acquisition.
type: boolean
WaterSuppressionTechnique:
name: WaterSuppressionTechnique
display_name: Water Suppression Technique
description: |
The name of the pulse sequence used for water suppression (for example, `"CHESS"`, `"VAPOR"`).
type: string
WholeBloodAvail:
name: WholeBloodAvail
display_name: Whole Blood Avail
Expand Down
6 changes: 6 additions & 0 deletions src/schema/rules/sidecars/mri.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -30,10 +30,12 @@ MRIHardware:
level: recommended, but required for Arterial Spin Labeling
ReceiveCoilName: recommended
ReceiveCoilActiveElements: recommended
NumberReceiveCoilActiveElements: optional
GradientSetType: recommended
MRTransmitCoilSequence: recommended
MatrixCoilMode: recommended
CoilCombinationMethod: recommended
NumberTransmitCoilActiveElements: optional

MRISample:
selectors:
Expand Down Expand Up @@ -77,6 +79,10 @@ MRISequenceSpecifics:
SpoilingRFPhaseIncrement: optional
SpoilingGradientMoment: optional
SpoilingGradientDuration: optional
WaterSuppression: optional
WaterSuppressionTechnique: optional
Comment on lines +82 to +83
Copy link
Contributor

@markmikkelsen markmikkelsen Feb 29, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

@effigies DICOM tag (0018,9025) fits these, but the value is not boolean as is proposed in #1377. The DICOM tag also covers OuterVolumeSuppression that the MRS BEP includes (which would be the FAT value in DICOM).

Copy link
Collaborator

Choose a reason for hiding this comment

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

So do we want to add something like the following to WaterSuppression:

Corresponds to DICOM Tag 0018, 9025 SpectrallySelectedSuppression.
If present and not NONE, then this value SHOULD be True, otherwise False.

And the following to WaterSuppressionTechnique?

Corresponds to DICOM Tag 0018, 9025 SpectrallySelectedSuppression.
May be omitted if the value is NONE.

Copy link
Contributor

Choose a reason for hiding this comment

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

Makes sense to me! And the same would be the case for OuterVolumeSuppression.

B0ShimmingTechnique: optional
B1ShimmingTechnique: optional

PETMRISequenceSpecifics:
selectors:
Expand Down