Integrate TF at tensorflow/tensorflow@1914c410b274#3549
Closed
GMNGeoffrey wants to merge 8 commits intoiree-org:googlefrom
Closed
Integrate TF at tensorflow/tensorflow@1914c410b274#3549GMNGeoffrey wants to merge 8 commits intoiree-org:googlefrom
GMNGeoffrey wants to merge 8 commits intoiree-org:googlefrom
Conversation
Updates TF to [1914c410b274](tensorflow/tensorflow@101df483a5d4) matching [f402e682d0ef](llvm/llvm-project@f402e682d0ef). Updates Docker images to use Python3 instead of Python2 since TF looks for Python3 first as of [ddb837a81022](tensorflow/tensorflow@ddb837a81022).
copybara-service Bot
pushed a commit
that referenced
this pull request
Oct 20, 2020
Updates TF to [1914c410b274](tensorflow/tensorflow@101df483a5d4) matching [f402e682d0ef](llvm/llvm-project@f402e682d0ef). Updates Docker images to use Python3 instead of Python2 since TF looks for Python3 first as of [ddb837a81022](tensorflow/tensorflow@ddb837a81022). It's still annoying that we need numpy to run our C++ code, but at least it's modern numpy now. See #1737. We have to leave the `bazel-tensorflow-nvidia` image pinned at an old version because when we rerun `apt-get install` on the nvidia driver something gets screwed up. See #3550 COPYBARA_INTEGRATE_REVIEW=#3549 from GMNGeoffrey:tf-py3 3ca1099 PiperOrigin-RevId: 338161374
Contributor
Author
|
Merged as 7411cba |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates TF to
1914c410b274
matching
f402e682d0ef.
Updates Docker images to use Python3 instead of Python2 since TF looks
for Python3 first as of
ddb837a81022. It's
still annoying that we need numpy to run our C++ code, but at least
it's modern numpy now. See #1737.
We have to leave the
bazel-tensorflow-nvidiaimage pinned at an oldversion because when we rerun
apt-get installon the nvidia driversomething gets screwed up. See
#3550