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

support for cv::VideoCapture #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

support for cv::VideoCapture #2

wants to merge 1 commit into from

Conversation

D-Alex
Copy link
Contributor

@D-Alex D-Alex commented Feb 16, 2016

VideoCapture is a generic camera interface from opencv supporting
cameras, video streams and video files. This is a basic
implementation for using VideoCapture as camera_base

VideoCapture is a generic camera interface from opencv supporting
cameras, video streams and video files. This is a basic
implementaion for using VideoCapture as camera_base
@jmachowinski
Copy link

Hm, should this not go into a new package then ?

@D-Alex
Copy link
Contributor Author

D-Alex commented Feb 16, 2016

I do not really like the tendency to generate a lot of repositories providing no functionality but just wrapper code. If someone wants to use VideoCapture without rock there is exactly no reason to use CamInterface. Therefore, the wrapper code is added here (CamInterface depends on opencv anyway)

@D-Alex D-Alex added the invalid label Feb 19, 2016
@D-Alex
Copy link
Contributor Author

D-Alex commented Feb 19, 2016

This is not going to compile with opencv < 3.0.0

@jmachowinski
Copy link

I'm more concerned about compile time. This package is included by every camera driver.

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

Successfully merging this pull request may close these issues.

2 participants