Skip to content
New issue

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

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

Already on GitHub? # to your account

expose apiPreference in VideoWriter #718

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

nibbleshift
Copy link

This pull request adds VideoWriter_OpenCap that accepts apiPrefrence.

To create a VideoWriter with apiPreference, the function VideoWriterCap has been exposed to accept the apiPreference. A test has been added to videoio_test to test gstreamer caps.

@nibbleshift
Copy link
Author

I am going to make some updates to this PR. I will look at gstreamer pipeline elements that are provided in the default install of gstreamer and remove the need for the h264 encoder.

The test I added does require gstreamer. Is this an issue in the AppVeyor configuration?

@golubaca
Copy link
Contributor

golubaca commented Aug 20, 2020

I think that opencv is not built with gstreamer inside appveyor. It uses appveyor.yml which calls appveyor_build_opencv.cmd for compiling opencv, and there is nothing about gstreamer.

@jackesdavid
Copy link

one question , can this pr still be merged?

Is there other way to achive this visbility to create VideoWriters to gstreamer?

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants