-
Notifications
You must be signed in to change notification settings - Fork 2.8k
FEATURE: New Video
API
#1924
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.
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
base: develop
Are you sure you want to change the base?
FEATURE: New Video
API
#1924
Conversation
process_video
Hi @Ashp116 👋🏻 Another great idea! Video processing is probably the oldest part of |
Hi @SkalskiP, yea, I would like to help update the API. I was thinking of changing how videos are written in |
Hi @Ashp116 I'm really glad you want to help me! Let's goooo! 🔥 🔥 🔥 I want the functionalities currently found in
|
UPDATE: Added a new Video class with OpenCV writer and backend
…supervision into bug/process-video-audio
Hi @SkalskiP, I’ve addressed most of the features you mentioned, but I have some thoughts on a few aspects of the implementation:
|
process_video
Video
class
Hi @SkalskiP, Thank you for reviewing the PR. I have addressed all the comments from the review. Could you please take a look at the following points?
Please let me know if you have any feedback or suggestions. Thanks |
Hi @SkalskiP, It’s been a while! I’ve added better audio support. Previously, I manually manipulated audio packets, along with I’ve included my Colab notebook showcasing the new I’d love to hear your thoughts and get your feedback on my current implementation. Thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Improve Hight quality video web
Description
This PR introduces a new
Video
API that streamlines video processing and rendering workflows. It addresses both issues #1923 and #1929 by enabling more flexible backend support and improved audio-video synchronization.With this update, the video processing function now supports multiple backends, including
PyAV
andOpenCV
. Notably,PyAV
is the only backend currently supporting audio rendering, which significantly improves output quality.This PR requires the optional dependency
pyAV
for the video rendering backend.Tags:
Fixes #1923
Fixes #1929
Type of change
How has this change been tested, please provide a testcase or example of how you tested the change?
Please refer to #1923 and #1929
Any specific deployment considerations
Ensure that
pyAV
is installed in the environment to testpyAV
backend.Docs