Merge pull request #175 from NeurodataWithoutBorders/fix/only_two_timestamps

make timeseries with < 3 timestamps exempt from regular_timestamps check

Author: CodyCBakerPhD

nwbinspector/checks/time_series.py

@@ -13,8 +13,10 @@ def check_regular_timestamps(
     time_series: TimeSeries, time_tolerance_decimals: int = 9, gb_severity_threshold: float = 1.0
 ):
     """If the TimeSeries uses timestamps, check if they are regular (i.e., they have a constant rate)."""
-    if time_series.timestamps is not None and check_regular_series(
-        series=time_series.timestamps, tolerance_decimals=time_tolerance_decimals
+    if (
+        time_series.timestamps is not None
+        and len(time_series.timestamps) > 2
+        and check_regular_series(series=time_series.timestamps, tolerance_decimals=time_tolerance_decimals)
     ):
         timestamps = np.array(time_series.timestamps)
         if timestamps.size * timestamps.dtype.itemsize > gb_severity_threshold * 1e9:

tests/unit_tests/test_time_series.py

@@ -35,6 +35,21 @@ def test_check_regular_timestamps():
     )
 
 
+def test_pass_check_regular_timestamps():
+    """Should pass because there are only two timestamps"""
+    assert (
+        check_regular_timestamps(
+            time_series=pynwb.TimeSeries(
+                name="test_time_series",
+                unit="test_units",
+                data=[0, 0],
+                timestamps=[1.2, 3.2],
+            )
+        )
+        is None
+    )
+
+
 def test_check_data_orientation():
     assert check_data_orientation(
         time_series=pynwb.TimeSeries(