search

openvinotoolkit / openvino

OpenVINO™ is an open-source toolkit for optimizing and deploying AI inference
https://docs.openvino.ai
Apache License 2.0
6.79k stars 2.16k forks source link

Use cldnn from system if available. #118

Closed tripzero closed 4 years ago

tripzero commented 5 years ago

Right now you cannot compile the GPU device plugin without ENABLE_CLDNN and ENABLE_CLDNN automatically tries to build CLDNN from thirdparty. CLDNN plugin should try to use CLDNN installed separately on system before using thirdparty/cldnn.

ilya-lavrenov commented 5 years ago

Why do you think so? CLDNN which is a part of DLDT source tree is more fresh and optimised.

tripzero commented 5 years ago

What if another project uses cldnn? Now you have two versions installed for no reason. If there are non-upstream optimizations to cldnn, they should be made part of the upstream otherwise other projects will not benefit. Likewise, fixes that happen upstream will not be automatically pulled into dldt's fork without extra efforts.

Another project that does this is ngraph. If you have dldt and ngraph on same system, you have two different trees of cldnn and two different installs. This will cause problems during link/runtime.

vladimir-paramuzov commented 4 years ago

CLDNN is not supported anymore as a standalone project. So the issue seems to be not relevant anymore.