search

finalfusion / finalfusion-tensorflow-ops

Tensorflow Op for finalfusion embeddings
https://finalfusion.github.io/
Other
2 stars 0 forks source link

Remove tensorflow requires. #15

Closed sebpuetz closed 5 years ago

sebpuetz commented 5 years ago

Can lead to mismatch between the tf distribution returning the compilation flags etc. and the one installed by setup.py.

If e.g. tensorflow-rocm is installed, import tensorflow in the cmake call will import tensorflow-rocm which is not necessarily compiled in the same manner as tensorflow which is installed as a dependency by the setup.

@twuebi ran into ABI incompatibility issues here since tensorflow-rocm is compiled with a different CXX_11_ABI flag than the official tensorflow wheels.