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

Support tensorflow (purelib) #15

Open
kerinin opened this issue Apr 26, 2019 · 3 comments
Open

Support tensorflow (purelib) #15

kerinin opened this issue Apr 26, 2019 · 3 comments

Comments

@kerinin
Copy link

kerinin commented Apr 26, 2019

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

@apt-itude
Copy link
Owner

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.

@kerinin
Copy link
Author

kerinin commented May 23, 2019

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.

@apt-itude
Copy link
Owner

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.

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

No branches or pull requests

2 participants