Skip to content

FFMPEG 16bit file support#12284

Closed
shawndfernandes wants to merge 2 commits intoopencv:masterfrom
shawndfernandes:patch-1
Closed

FFMPEG 16bit file support#12284
shawndfernandes wants to merge 2 commits intoopencv:masterfrom
shawndfernandes:patch-1

Conversation

@shawndfernandes
Copy link
Copy Markdown

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
Copy Markdown
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
Copy Markdown
Author

Tested functionality with FFV1 lossless codec.

Added tests in #12290

@mshabunin
Copy link
Copy Markdown
Contributor

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

@shawndfernandes
Copy link
Copy Markdown
Author

Changes and tests done in #12290 12290

@shawndfernandes shawndfernandes deleted the patch-1 branch August 25, 2018 20:35
@AlexanderKhazatsky
Copy link
Copy Markdown

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.

4 participants