Skip to content

Commit

Permalink
Recommend SliceTiming if MRAcquisitionType is "2D" (#1594)
Browse files Browse the repository at this point in the history
* Recommend SliceTiming if MRAcquisitionType is "2D"

* SliceEncodingDirection is rec only if 2D.
  • Loading branch information
tsalo authored Sep 22, 2023
1 parent b768f5b commit 466e212
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -98,7 +98,7 @@ The definitions of the fields specified in these tables may be found in
A guide for using macros can be found at
https://github.com/bids-standard/bids-specification/blob/master/macros_doc.md
-->
{{ MACROS___make_sidecar_table("mri.MRITimingParameters") }}
{{ MACROS___make_sidecar_table(["mri.MRITimingParameters", "mri.SliceTimingMRI"]) }}

### RF & Contrast

Expand Down
10 changes: 8 additions & 2 deletions src/schema/rules/sidecars/mri.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -187,13 +187,19 @@ MRITimingParameters:
should only be used when the volume timing is critical for interpretation
of the data, such as in ASL or variable echo time fMRI sequences.
InversionTime: recommended
DwellTime: recommended

SliceTimingMRI:
selectors:
- modality == "mri"
- sidecar.MRAcquisitionType == "2D"
fields:
SliceTiming:
level: recommended
level_addendum: |
required for sparse sequences that do not have the `DelayTime` field set,
and Arterial Spin Labeling with `MRAcquisitionType` set on `2D`.
SliceEncodingDirection: recommended
DwellTime: recommended

SliceTimingASL:
selectors:
Expand All @@ -207,7 +213,7 @@ SliceTimingASL:
issue:
code: SLICE_TIMING_NOT_DEFINED_2D_ASL
message: |
You should define `SliceTiming` for this file, because `SequenceType` is sets
You should define `SliceTiming` for this file, because `SequenceType` is set
to a 2D sequence. `SliceTiming` is the time at which each slice was
acquired within each volume (frame) of the acquisition. Slice timing
is not slice order -- rather, it is a list of times containing the
Expand Down

0 comments on commit 466e212

Please sign in to comment.