rules_pip icon indicating copy to clipboard operation
rules_pip copied to clipboard

Support tensorflow (purelib)

Open kerinin opened this issue 5 years ago • 3 comments

A similar problem existed in rules_python (https://github.com/bazelbuild/rules_python/issues/71) but it appears to be resolved. I would really like to use rules_pip but I need tensorflow.

kerinin avatar Apr 26 '19 20:04 kerinin

Hi @kerinin, thanks for bringing this up. I definitely should add support for purelib/platlib distributions, so I will look into this when I get a chance. It actually doesn't look like the issue you referenced or its root cause (bazelbuild/rules_python#92) have been resolved, although others have posted workarounds specifically for tensorflow.

Given that tensorflow actually is a Bazel project, have you tried depending on it via the GitHub repository instead? They appear to have a public //tensorflow:tensorflow_py target for the Python library.

apt-itude avatar May 02 '19 15:05 apt-itude

It may be possible to build tensorflow as a remote repository, but building it takes about 6h, which isn't really a workable solution for my team.

kerinin avatar May 23 '19 18:05 kerinin

Oh wow, I didn't realize the build time was that high. If you used a shared remote build cache you'd only need to do it once until you need to upgrade. Either way, I do need to add support for this.

apt-itude avatar May 24 '19 02:05 apt-itude