-
Notifications
You must be signed in to change notification settings - Fork 86
Conversation
This PR needs Approvals as follows.
Please choose reviewers and requet reviews! Click to see how to approve each reviewsYou can approve this PR by triggered comments as follows.
See all trigger commentsPlease replace [Target] to review target
|
@oatawa1 |
@iizukak I thought the 0.8.1 could work so. I checked it using # pip check
tensorflow-metadata 0.23.0 requires absl-py, which is not installed.
tensorflow-gpu 1.15.3 requires absl-py, which is not installed.
tensorflow-datasets 1.3.0 requires absl-py, which is not installed.
tensorboard 1.15.0 requires absl-py, which is not installed. After I installed the absl-py. absl-py==0.9.0 # pip check
tensorflow-metadata 0.23.0 has requirement absl-py<0.9,>=0.7, but you have absl-py 0.9.0. absl-py==0.8.1 # pip check
No broken requirements found. |
@oatawa1 Great. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OA
/ready |
⏳Merge job is queued... |
In our case, the |
What this patch does to fix the issue.
This PR fix absl-py version to 0.8.1 (the latest version before 0.9)
Because absl-py require <0.9, >=0.7
Note that Blueoil code didn't require
absl-py
but it required by TensorFlow.Link to any relevant issues or pull requests.
#1167