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

Opencv io support #1061

Closed
bhack opened this issue Sep 10, 2014 · 6 comments
Closed

Opencv io support #1061

bhack opened this issue Sep 10, 2014 · 6 comments

Comments

@bhack
Copy link
Contributor

bhack commented Sep 10, 2014

@shelhamer I want to re-add support to opencv mat and vector of mat removed from #995 here https://github.com/kloudkl/caffe/commit/725e98d32d2fe5caf4c065cd1241d3622092b809.
@kloudkl Do you have already any plan to open a new PR?

@kloudkl
Copy link
Contributor

kloudkl commented Sep 10, 2014

ReadImageToDatum is enough for most users whose images live on their local disks. Maybe cf99705 is still helpful.

@bhack
Copy link
Contributor Author

bhack commented Sep 10, 2014

ReadImageToDatum is not the solution I.e. for realtime camera application when you use the network for CPU or GPU forward pass. I want invest time on this also if we need to find another solution that doesn't rely on removed commit. Who can give us a feedback?

@shelhamer
Copy link
Member

@bhack please do try to add support but know that it can't conflict with
nvcc and break the build for gcc / clang and linux / osx like it did in the
earlier PR.

On Wed, Sep 10, 2014 at 9:56 AM, bhack notifications@github.com wrote:

@shelhamer https://github.com/shelhamer I want to re-add support to
opencv mat and vector of mat removed from #995
#995.
@kloudkl https://github.com/kloudkl Do you have already any plan to
open a new PR?


Reply to this email directly or view it on GitHub
#1061.

@bhack
Copy link
Contributor Author

bhack commented Sep 10, 2014

@shelhamer I know but It is hard to solve osx problem without an OSX machine or build server. How we can handle this?

@bhack
Copy link
Contributor Author

bhack commented Sep 11, 2014

Take a look at #1068

@shelhamer
Copy link
Member

This is effectively solved by #1070. The NVCC compatibility fix in #1236 might make this ok for OS X and and legacy CUDA too (not that I recommend trying to make legacy CUDA work -- 6.5 is our official suggestion).

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

No branches or pull requests

3 participants