search

tensorflow / ngraph-bridge

TensorFlow-nGraph bridge
Other
137 stars 65 forks source link

Master compiled with --build-plaidml_backend option will not display PLAIDML in list of backends. #582

Open XReyRobert-IBM opened 4 years ago

XReyRobert-IBM commented 4 years ago

Argument is properly passed to cmake though...

Running COMMAND: cmake -DNGRAPH_INSTALL_PREFIX=/Users/xav/MyDev/python/ngraph-bridge-latest/ngraph-bridge/build_cmake/artifacts -DNGRAPH_USE_CXX_ABI=0 -DNGRAPH_DEX_ONLY=TRUE -DNGRAPH_DEBUG_ENABLE=NO -DNGRAPH_UNIT_TEST_ENABLE=NO -DNGRAPH_TARGET_ARCH=native -DNGRAPH_TUNE_ARCH=native -DNGRAPH_TBB_ENABLE=FALSE -DNGRAPH_TOOLS_ENABLE=NO -DNGRAPH_GPU_ENABLE=NO -DNGRAPH_PLAIDML_ENABLE=YES -DNGRAPH_INTELGPU_ENABLE=NO /Users/xav/MyDev/python/ngraph-bridge-latest/ngraph-bridge/build_cmake/ngraph

update: I noticed this during build of ngraph:

CMake Warning: Manually-specified variables were not used by the project:

NGRAPH_INTELGPU_ENABLE
NGRAPH_PLAIDML_ENABLE
NGRAPH_USE_CXX_ABI
XReyRobert-IBM commented 4 years ago

Well... I sadly think I have the answer :/ Plaidml was simply droped from ngraph...

https://github.com/NervanaSystems/ngraph/pull/4554

diyessi commented 4 years ago

PlaidML switched to MLIR; ngraph has not actually worked with what has been the current version of PlaidML for some time, the above PR just acknowledged the reality from the ngraph side.