New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature: Enhanced Video Processor with Keyframe Detection #1

Open

davidj-brewster wants to merge 1 commit into main from pr-branch

Owner

davidj-brewster commented Nov 10, 2024

Changes

Add async video processing with batched frame handling
Implement configurable keyframe detection using SSIM
Add progress tracking and callback support
Improve error handling and logging
Add comprehensive type hints and documentation

Testing

Unit tests added for core functionality
Tested with various video formats
Verified memory usage with large files

Review Notes

Configuration parameters may need tuning
Consider adding more logging points
May want to review batch size defaults


          gh-copilot suggested changes

1c65508

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet