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
fix bug in output format for pyav #6672
fix bug in output format for pyav #6672
Conversation
Tested locally: With default parameters: video = read_video("/Users/jdsgomes/repos/video_reader_benchmark/videos/R6llTwEh07w.mp4")
video[0].shape torch.Size([303, 256, 340, 3]) With output_format="TCHW" video = read_video("/Users/jdsgomes/repos/video_reader_benchmark/videos/R6llTwEh07w.mp4", output_format="TCHW")
video[0].shape torch.Size([303, 3, 256, 340]) |
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.
This looks good to me.
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.
Thanks @jdsgomes , the PR looks good to me! Lets cherry pick this for v0.14 release.
Hey @jdsgomes! You merged this PR, but no labels were added. The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py |
* fix bug in output format for pyav * add read from memory with constructor overload * Revert "add read from memory with constructor overload" This reverts commit 14cbbab. * run ufmt
* fix bug in output format for pyav * add read from memory with constructor overload * Revert "add read from memory with constructor overload" This reverts commit 14cbbab. * run ufmt
Summary: * fix bug in output format for pyav * add read from memory with constructor overload * Revert "add read from memory with constructor overload" This reverts commit 14cbbab. * run ufmt Reviewed By: datumbox Differential Revision: D40138748 fbshipit-source-id: f9104cf9c56c3187a22ca6156267d9bf23552f3f
fixes bug:
when using
video_reader
backend,read_video
ignores theoutput_format
parameter