[tests] Debug release test failures on MacOS hardware

Author: Breakthrough

.github/workflows/release-test.yml

@@ -76,7 +76,15 @@ jobs:
           pip install .[opencv,pyav,moviepy]
           pip install opentimelineio pillow psutil pytest
       - name: Run release tests
-        run: pytest -m release -vv --ignore=tests/release/test_long_video_stress.py --ignore=tests/release/test_install_matrix.py
+        env:
+          SCENEDETECT_DEBUG_LOG: scenedetect-debug-${{ matrix.os }}-${{ matrix.python-version }}.log
+        run: pytest -m release -vv -k "Histogram or Adaptive" --ignore=tests/release/test_long_video_stress.py --ignore=tests/release/test_install_matrix.py
+      - name: Upload detector debug log
+        if: always()
+        uses: actions/upload-artifact@v4
+        with:
+          name: scenedetect-debug-${{ matrix.os }}-${{ matrix.python-version }}
+          path: scenedetect-debug-${{ matrix.os }}-${{ matrix.python-version }}.log
 
   long-stress:
     needs: static

scenedetect/detectors/adaptive_detector.py

@@ -137,6 +137,19 @@ def process_frame(self, timecode: FrameTimecode, frame_img: np.ndarray) -> list[
             adaptive_ratio >= self.adaptive_threshold and target_score >= self.min_content_val
         )
         min_length_met: bool = (timecode - self._last_cut) >= self.min_scene_len
+        logger.debug(
+            "[AdaptiveDetector] target_frame=%d adaptive_ratio=%.10f adaptive_threshold=%.10f "
+            "target_score=%.10f min_content_val=%.10f avg_window=%.10f threshold_met=%s "
+            "min_length_met=%s",
+            target_timecode.frame_num,
+            adaptive_ratio,
+            self.adaptive_threshold,
+            target_score,
+            self.min_content_val,
+            average_window_score,
+            threshold_met,
+            min_length_met,
+        )
         if threshold_met and min_length_met:
             self._last_cut = target_timecode
             return [target_timecode]

scenedetect/detectors/histogram_detector.py

@@ -16,13 +16,16 @@
 """
 
 import typing as ty
+from logging import getLogger
 
 import cv2
 import numpy
 
 from scenedetect.common import FrameTimecode
 from scenedetect.detector import SceneDetector
 
+logger = getLogger("pyscenedetect")
+
 
 class HistogramDetector(SceneDetector):
     """Compares the difference in the Y channel of YUV histograms for adjacent frames. When the
@@ -104,9 +107,18 @@ def process_frame(self, timecode: FrameTimecode, frame_img: numpy.ndarray) -> li
             # Example: If `_threshold` is set to 0.8, it implies that only changes resulting in a
             # correlation less than 0.8 between histograms will be considered significant enough to
             # denote a scene change.
-            if hist_diff <= self._threshold and (
-                (timecode - self._last_cut) >= self._min_scene_len
-            ):
+            cut_threshold_met = hist_diff <= self._threshold
+            min_len_met = (timecode - self._last_cut) >= self._min_scene_len
+            logger.debug(
+                "[HistogramDetector] frame=%d hist_diff=%.10f threshold=%.10f "
+                "threshold_met=%s min_len_met=%s",
+                timecode.frame_num,
+                hist_diff,
+                self._threshold,
+                cut_threshold_met,
+                min_len_met,
+            )
+            if cut_threshold_met and min_len_met:
                 cut_list.append(timecode)
                 self._last_cut = timecode
 

tests/release/conftest.py

@@ -11,6 +11,7 @@
 #
 """Fixtures for the release test suite."""
 
+import logging
 import os
 
 import pytest
@@ -22,6 +23,23 @@
     generate_vfr_swing,
 )
 
+
+def pytest_configure(config):
+    # Force scenedetect import so init_logger() runs (it wipes logger handlers
+    # at module load); without this our handler gets removed at first import.
+    import scenedetect as _scenedetect
+
+    _ = _scenedetect
+
+    pyscenedetect_logger = logging.getLogger("pyscenedetect")
+    pyscenedetect_logger.setLevel(logging.DEBUG)
+    log_path = os.path.abspath(os.environ.get("SCENEDETECT_DEBUG_LOG", "scenedetect-debug.log"))
+    handler = logging.FileHandler(log_path, mode="w")
+    handler.setLevel(logging.DEBUG)
+    handler.setFormatter(logging.Formatter("%(levelname)s %(message)s"))
+    pyscenedetect_logger.addHandler(handler)
+
+
 REPO_ROOT = os.path.dirname(os.path.dirname(os.path.dirname(os.path.abspath(__file__))))