Skip to content
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

FFMPEG 16bit file support #12284

Closed
wants to merge 2 commits into from
Closed

Conversation

shawndfernandes
Copy link

Added support for FFMPEG 16 bit channel support.

Resolves VideoWriter support for 16-bit depth images and color-alpha color imags #10623

This pullrequest changes

Added support for FFMPEG to write 16 bit RGB depth video along with 8bit RGB

Added support for FFMPEG 16 bit channel support.
@mshabunin
Copy link
Contributor

Did you check CvVideoWriter_FFMPEG::writeFrame function? It accepts only two pixel formats: AV_PIX_FMT_BGR24 and AV_PIX_FMT_GRAY8.

I beleive we need a test for this functionality, check modules/videoio/test/test_ffmpeg.cpp file for FFmpeg specific tests.

shawndfernandes added a commit to shawndfernandes/opencv that referenced this pull request Aug 23, 2018
Added lossless FFV1 codec, and test for 16-bit 3 channel video writing
@shawndfernandes
Copy link
Author

Tested functionality with FFV1 lossless codec.

Added tests in #12290

@mshabunin
Copy link
Contributor

@shawndfernandes , you should add test to this PR, so that the new functionality would get tested automatically.

@shawndfernandes
Copy link
Author

Changes and tests done in #12290 12290

@AlexanderKhazatsky
Copy link

Hey! Is it possible to merge this into master? It would be extremely useful for encoding depth videos.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: videoio pr: needs test New functionality requires minimal tests set
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants