[lint] Add strict=True to all zip() calls

Author: Breakthrough

docs/changelog.md

@@ -393,4 +393,4 @@ This version of DVR-Scan includes a new, faster background subtraction algorithm
 
  * [general] Minimum Python version is now 3.10 ([EOL](https://devguide.python.org/versions/))
  * [bugfix] Fix `quiet-mode` setting (`-q`/`--quiet` flag) still allowing extraneous output
- * [bugfix] Fix loading config from file in UI
+ * [bugfix] Fix loading config from file in UI ([#252](https://github.com/Breakthrough/DVR-Scan/issues/252))

dvr_scan/overlays.py

@@ -102,7 +102,7 @@ def draw(self, frame: numpy.ndarray, text: str):
             x_offset = max(0, frame.shape[1] - (self._margin + self._border + max_width))
         y_offset = self._margin + self._border
 
-        for text, size in zip(lines, sizes):
+        for text, size in zip(lines, sizes, strict=True):
             text_pos = (x_offset, y_offset + size[0][1])
             cv2.putText(
                 frame,

dvr_scan/region.py

@@ -36,7 +36,9 @@ def __init__(self, value: str):
         if not len(values) >= 6:
             raise ValueError(f"Regions must have at least 3 points.\n  Input: {value}")
 
-        self._value = [Point(int(x), int(y)) for x, y in zip(values[0::2], values[1::2])]
+        self._value = [
+            Point(int(x), int(y)) for x, y in zip(values[0::2], values[1::2], strict=True)
+        ]
 
     @property
     def value(self) -> ty.List[Point]:

dvr_scan/shared/__init__.py

@@ -283,5 +283,5 @@ def logging_redirect_tqdm(
             ] + [tqdm_handler]
         yield
     finally:
-        for logger, original_handlers in zip(loggers, original_handlers_list):
+        for logger, original_handlers in zip(loggers, original_handlers_list, strict=True):
             logger.handlers = original_handlers

tests/test_scan_context.py

@@ -164,7 +164,7 @@ def test_pre_event_shift_with_frame_skip(traffic_camera_video):
         assert all(
             [
                 abs(x[0] - y[0]) <= frame_skip
-                for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_PRE_5)
+                for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_PRE_5, strict=True)
             ]
         ), "Comparison failure when frame_skip = %d" % (frame_skip)
 
@@ -195,13 +195,16 @@ def test_post_event_shift_with_frame_skip(traffic_camera_video):
         assert all(
             [
                 abs(x[1] - y[1]) <= frame_skip
-                for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_POST_40)
+                for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_POST_40, strict=True)
             ]
         ), "Comparison failure when frame_skip = %d" % (frame_skip)
         # The calculated end times must always be >= the ground truth's frame number, otherwise
         # we may be discarding frames containing motion due to skipping them.
         assert all(
-            [x[1] >= y[1] for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_POST_40)]
+            [
+                x[1] >= y[1]
+                for x, y in zip(event_list, TRAFFIC_CAMERA_EVENTS_TIME_POST_40, strict=True)
+            ]
         ), "Comparison failure when frame_skip = %d" % (frame_skip)