Standardize min scene length message across detectors by Janscode · Pull Request #438 · Breakthrough/PySceneDetect
Navigation Menu
{{ message }}
Breakthrough / PySceneDetect Public
- Notifications You must be signed in to change notification settings
- Fork 481
Merged
Breakthrough merged 1 commit intoBreakthrough:mainfrom
Oct 19, 2024Merged
Standardize min scene length message across detectors#438
Breakthrough merged 1 commit intoBreakthrough:mainfrom
Standardize min scene length message across detectors#438
Breakthrough merged 1 commit intoBreakthrough:mainfrom
Conversation
Copy link Copy Markdown
Contributor
I found the min_scene_length docs to be a little confusing for the AdaptiveDetector class, I think it's helpful to know 1) the value is in frames and 2) scenes won't be filtered out / dropped, instead they are merged
Breakthrough
self-assigned this
Copy link Copy Markdown
Owner
Breakthrough
commented
Oct 16, 2024
Breakthrough commented
Oct 16, 2024Thanks for improving this.
Breakthrough approved these changes Oct 16, 2024
Breakthrough
added this to the
0.6.5 milestone
Breakthrough
merged commit
7c79872
into
Breakthrough:main
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
2 participants