-
Notifications
You must be signed in to change notification settings - Fork 747
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
Added inference engine dnn model support #820
base: master
Are you sure you want to change the base?
Conversation
It's not a problem with security or anything, it's just Mac being annoying. |
It looks like those flags don't do anything unless we have OpenVINO already installed. I can merge this, but the binaries distributed are not going to contain any inference engine. That's alright with you? |
Ok, maybe I need a bit more help here, the library structure is the following:
Now Following your example, I would have to specify the rpath realtive to the And what does this stand for
|
BTW, "nGraph has moved to OpenVINO", so once we get presets for OpenVINO in place, we can get rid of the old ones for nGraph. /cc @EmergentOrder |
@saudet Well with the flag it worked when you had the inference engine installed locally. This is at least something, but yeah of course, shipping the IE would be way better. Do you think it would make more sense to create a preset for the inference engine itself? Have you already looked into that? |
Haven't looked into it, but the high-level API looks simple enough and shouldn't be too hard to map: |
As discussed in bytedeco/javacv#1344 I have added the support for the Intels Deep Learning Inference Engine. The compile.sh has been extended by the guide of the wiki:
https://github.com/opencv/opencv/wiki/Intel's-Deep-Learning-Inference-Engine-backend#build-opencv-from-source
There is still an error to (with a workaround) on MacOS. But this seems to be OS specific and a problem of the new System Integrity Protection: bytedeco/javacv#1344 (comment)