[Schema][Fix] add checks for multiple frame consistency issues (#2310)

* add checks for multiple frame consistency issues

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* keeping me honest yamllint

* remove redundant selector ||

* remove redundant check

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: bendhouseart

src/schema/rules/checks/pet.yaml

@@ -4,6 +4,20 @@
 PETFrameConsistency:
   issue:
     code: PET_FRAME_CONSISTENCY
+    message: |
+      The number of frames as defined by FrameDuration and FrameTimesStart do not match in the
+      associated '.json' file.
+    level: error
+  selectors:
+    - suffix == 'pet'
+    - sidecar.FrameDuration
+    - sidecar.FrameTimesStart
+  checks:
+    - length(sidecar.FrameDuration) == length(sidecar.FrameTimesStart)
+
+PETFrameConsistencyFrameDuration:
+  issue:
+    code: PET_FRAME_CONSISTENCY_FRAME_DURATION
     message: |
       The number of frames in this scan does not match the number of frames (as defined by FrameDuration) in the
       associated '.json' file.
@@ -14,3 +28,17 @@ PETFrameConsistency:
     - sidecar.FrameDuration
   checks:
     - length(sidecar.FrameDuration) == nifti_header.dim[4]
+
+PETFrameConsistencyFrameTimesStart:
+  issue:
+    code: PET_FRAME_CONSISTENCY_FRAME_TIMES_START
+    message: |
+      The number of frames in this scan does not match the number of frames (as defined by FrameTimesStart) in the
+      associated '.json' file.
+    level: error
+  selectors:
+    - suffix == 'pet'
+    - type(nifti_header) != "null"
+    - sidecar.FrameTimesStart
+  checks:
+    - length(sidecar.FrameTimesStart) == nifti_header.dim[4]