[ENH] Add FrameAcquisitionDuration

Author: effigies

src/modality-specific-files/magnetic-resonance-imaging-data.md

@@ -600,13 +600,13 @@ to be populated for functional sequences. Note that all these options can be
 used for non sparse sequences but that only options B, D and E are valid for
 sparse sequences.
 
-|          | **`RepetitionTime`** | **`SliceTiming`** | **`AcquisitionDuration`** | **`DelayTime`** | **`VolumeTiming`** |
-| -------- | -------------------- | ----------------- | ------------------------- | --------------- | ------------------ |
-| option A | \[ X ]               |                   | \[ ]                      |                 | \[ ]               |
-| option B | \[ ]                 | \[ X ]            |                           | \[ ]            | \[ X ]             |
-| option C | \[ ]                 |                   | \[ X ]                    | \[ ]            | \[ X ]             |
-| option D | \[ X ]               | \[ X ]            | \[ ]                      |                 | \[ ]               |
-| option E | \[ X ]               |                   | \[ ]                      | \[ X ]          | \[ ]               |
+|          | **`RepetitionTime`** | **`SliceTiming`** | **`FrameAcquisitionDuration`** | **`DelayTime`** | **`VolumeTiming`** |
+| -------- | -------------------- | ----------------- | ------------------------------ | --------------- | ------------------ |
+| option A | \[ X ]               |                   | \[ ]                           |                 | \[ ]               |
+| option B | \[ ]                 | \[ X ]            |                                | \[ ]            | \[ X ]             |
+| option C | \[ ]                 |                   | \[ X ]                         | \[ ]            | \[ X ]             |
+| option D | \[ X ]               | \[ X ]            | \[ ]                           |                 | \[ ]               |
+| option E | \[ X ]               |                   | \[ ]                           | \[ X ]          | \[ ]               |
 
 **Legend**
 

src/schema/objects/metadata.yaml

@@ -1199,6 +1199,15 @@ FlipAngle:
         unit: degree
         exclusiveMinimum: 0
         maximum: 360
+FrameAcquisitionDuration:
+  name: FrameAcquisitionDuration
+  display_name: Frame Acquisition Duration
+  description: |
+    Duration (in seconds) of volume acquisition.
+    Corresponds to DICOM Tag 0018, 9220 `Frame Acquisition Duration`.
+  type: number
+  exclusiveMinimum: 0
+  unit: s
 FrameDuration:
   name: FrameDuration
   display_name: Frame Duration

src/schema/rules/checks/func.yaml

@@ -94,18 +94,18 @@ VolumeTimingRepetitionTimeMutex:
   checks:
     - type(sidecar.RepetitionTime) == "null"
 
-RepetitionTimeAcquisitionDurationMutex:
+RepetitionTimeFrameAcquisitionDurationMutex:
   issue:
     code: REPETITION_TIME_AND_ACQUISITION_DURATION_MUTUALLY_EXCLUSIVE
     message: |
-      The fields 'RepetitionTime' and 'AcquisitionDuration' for this file are mutually exclusive.
+      The fields 'RepetitionTime' and 'FrameAcquisitionDuration' for this file are mutually exclusive.
       To specify acquisition duration, use 'SliceTiming' or 'DelayTime'
-      (RepetitionTime - AcquisitionDuration).
+      (RepetitionTime - FrameAcquisitionDuration).
     level: error
   selectors:
     - type(nifti_header) != "null"
     - intersects([suffix], ["asl", "bold"])
-    - type(sidecar.AcquisitionDuration) != "null"
+    - type(sidecar.FrameAcquisitionDuration) != "null"
   checks:
     - type(sidecar.RepetitionTime) == "null"
 
@@ -114,7 +114,7 @@ VolumeTimingDelayTimeMutex:
     code: VOLUME_TIMING_AND_DELAY_TIME_MUTUALLY_EXCLUSIVE
     message: |
       The fields 'VolumeTiming' and 'DelayTime' for this file are mutually exclusive.
-      To specify acquisition duration, use 'AcquisitionDuration' or 'SliceTiming'.
+      To specify acquisition duration, use 'FrameAcquisitionDuration' or 'SliceTiming'.
     level: error
   selectors:
     - type(nifti_header) != "null"
@@ -123,15 +123,15 @@ VolumeTimingDelayTimeMutex:
   checks:
     - type(sidecar.DelayTime) == "null"
 
-VolumeTimingMissingAcquisitionDuration:
+VolumeTimingMissingFrameAcquisitionDuration:
   issue:
     code: VOLUME_TIMING_MISSING_ACQUISITION_DURATION
     message: |
-      The field 'VolumeTiming' requires 'AcquisitionDuration' or 'SliceTiming' to be defined.
+      The field 'VolumeTiming' requires 'FrameAcquisitionDuration' or 'SliceTiming' to be defined.
     level: error
   selectors:
     - type(nifti_header) != "null"
     - intersects([suffix], ["asl", "bold"])
     - type(sidecar.VolumeTiming) != "null"
   checks:
-    - '"SliceTiming" in sidecar || "AcquisitionDuration" in sidecar'
+    - '"SliceTiming" in sidecar || "FrameAcquisitionDuration" in sidecar'

src/schema/rules/sidecars/func.yaml

@@ -44,7 +44,7 @@ MRIFuncVolumeTiming:
       description_addendum: |
         This field is mutually exclusive with `"DelayTime"`.
         If defined, this requires acquisition time (TA) be defined via either
-        `"SliceTiming"` or `"AcquisitionDuration"`.
+        `"SliceTiming"` or `"FrameAcquisitionDuration"`.
 
 # Timing Parameters
 MRIFuncTimingParameters:
@@ -56,8 +56,8 @@ MRIFuncTimingParameters:
     NumberOfVolumesDiscardedByScanner: recommended
     NumberOfVolumesDiscardedByUser: recommended
     DelayTime: recommended
-    AcquisitionDuration:
-      level: recommended
+    FrameAcquisitionDuration:
+      level: optional
       level_addendum: |
         required for sequences that are described with the `VolumeTiming`
         field and that do not have the `SliceTiming` field set to allow for