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

Feature Request: Include inference engine #214

Closed
ohlr opened this issue Jun 21, 2019 · 3 comments · Fixed by #235
Closed

Feature Request: Include inference engine #214

ohlr opened this issue Jun 21, 2019 · 3 comments · Fixed by #235

Comments

@ohlr
Copy link

ohlr commented Jun 21, 2019

To be able to use: cv2.dnn.readNetFromModelOptimizer openCV must be build with the Inference Engine. Are there any plans to add this feature?

There already exists a repo that does it opencv-python-inference-engine however, it only supports Ubuntu 18.04 and above

@native-api
Copy link
Contributor

native-api commented Jul 30, 2019

This looks like a heavy dependency (comparing https://pypi.org/project/opencv-python-inference-engine/#files and https://pypi.org/project/opencv-python-headless/#files suggests +8MB to archive size (probably more since the engine build has a few other modules disabled) so expect +x2-3 of that to the file size).

How big is the niche for it? And how homogeneous -- https://docs.openvinotoolkit.org/latest/_docs_IE_DG_inference_engine_intro.html suggests that the Engine lib has lots of optional modules of its own? If the required configuration would end up too niche, a custom build for your specific use case would probably be a better option.

@skvark
Copy link
Member

skvark commented Jul 30, 2019

I think we could make a modification to the setup.py so that if you run python setup.py install it builds OpenCV in some kind of default configuration depending on the system where it is run. Users could then build a custom version by appending Cmake flags to the python setup.py install command. We could also release sdist to PyPI. For CI purposes the setup.py file should accept some special flags to run the builds in special configurations specific to this repo. This is just an idea, but it would make for example local builds easier.

@native-api
Copy link
Contributor

Users could then build a custom version by appending Cmake flags to the python setup.py install command.

That's easy: #235

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

Successfully merging a pull request may close this issue.

3 participants