phc2sys, ts2phc missing samples

src/vse_sync_pp/analyzers/analyzer.py

@@ -144,6 +144,13 @@ def _timestamp_from_dec(self, dec):
         # relative time
         return dec
 
+    @staticmethod
+    def _check_missing_samples(data, result, reason):
+        if reason is None:
+            if len(data.timestamp.diff().astype(float).round(0).tail(-1).unique()) > 1:
+                return (False, "missing test samples")
+        return result, reason
+
     @property
     def result(self):
         """The boolean result from this analyzer's test of the collected data"""
@@ -260,8 +267,6 @@ def test(self, data):
             return (False, "short test duration")
         if len(data) - 1 < self._duration_min:
             return (False, "short test samples")
-        if len(data.timestamp.diff().astype(float).round(0).tail(-1).unique()) > 1:
-            return (False, "missing test samples")
         return (True, None)
 
     def explain(self, data):

src/vse_sync_pp/analyzers/phc2sys.py

@@ -12,17 +12,24 @@ class TimeErrorAnalyzer(TimeErrorAnalyzerBase):
     id_ = 'phc2sys/time-error'
     parser = id_
     locked = frozenset({'s2'})
-
+
+    def test(self, data):
+        return self._check_missing_samples(data, *super().test(data))
 
 class TimeDeviationAnalyzer(TimeDeviationAnalyzerBase):
     """Analyze time deviation"""
     id_ = 'phc2sys/time-deviation'
     parser = 'phc2sys/time-error'
     locked = frozenset({'s2'})
-
+
+    def test(self, data): 
+        return self._check_missing_samples(data, *super().test(data))
 
 class MaxTimeIntervalErrorAnalyzer(MaxTimeIntervalErrorAnalyzerBase):
     """Analyze max time interval error"""
     id_ = 'phc2sys/mtie'
     parser = 'phc2sys/time-error'
     locked = frozenset({'s2'})
+
+    def test(self, data): 
+        return self._check_missing_samples(data, *super().test(data))

src/vse_sync_pp/analyzers/ts2phc.py

@@ -11,17 +11,24 @@ class TimeErrorAnalyzer(TimeErrorAnalyzerBase):
     id_ = 'ts2phc/time-error'
     parser = id_
     locked = frozenset({'s2'})
-
+
+    def test(self, data): 
+        return self._check_missing_samples(data, *super().test(data))
 
 class TimeDeviationAnalyzer(TimeDeviationAnalyzerBase):
     """Analyze time deviation"""
     id_ = 'ts2phc/time-deviation'
     parser = 'ts2phc/time-error'
     locked = frozenset({'s2'})
 
+    def test(self, data): 
+        return self._check_missing_samples(data, *super().test(data)) 
 
 class MaxTimeIntervalErrorAnalyzer(MaxTimeIntervalErrorAnalyzerBase):
     """Analyze max time interval error"""
     id_ = 'ts2phc/mtie'
     parser = 'ts2phc/time-error'
     locked = frozenset({'s2'})
+
+    def test(self, data): 
+        return self._check_missing_samples(data, *super().test(data))