Skip to content

Fix AdaptiveDetector min_scene_len target frame check#557

Draft
aouxwoux wants to merge 1 commit into
Breakthrough:mainfrom
aouxwoux:codex/adaptive-min-scene-target-frame
Draft

Fix AdaptiveDetector min_scene_len target frame check#557
aouxwoux wants to merge 1 commit into
Breakthrough:mainfrom
aouxwoux:codex/adaptive-min-scene-target-frame

Conversation

@aouxwoux

@aouxwoux aouxwoux commented Jul 5, 2026

Copy link
Copy Markdown

Summary

  • apply AdaptiveDetector's min_scene_len check to the buffered target frame instead of the delayed/current frame
  • add a synthetic regression test for two adaptive spikes that are only four target frames apart

Fixes #408

Verification

  • python -m pytest tests/test_detectors.py::test_adaptive_detector_uses_target_frame_for_min_scene_len tests/test_detectors.py -q
  • python -m ruff check scenedetect/detectors/adaptive_detector.py tests/test_detectors.py
  • git diff --check

@Breakthrough Breakthrough force-pushed the main branch 2 times, most recently from ab2d6f2 to c1fcb79 Compare July 6, 2026 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

min_scene_len parameter isn't respected with AdaptiveDetector

1 participant