Skip to content

Move tensorflow commit to pick up shape inference API. #26348

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 30, 2019
Merged

Move tensorflow commit to pick up shape inference API. #26348

merged 2 commits into from
Jul 30, 2019

Conversation

bgogul
Copy link
Contributor

@bgogul bgogul commented Jul 25, 2019 •
edited
Loading

Moves the tensorflow commit to tensorflow/tensorflow@fee0b38d so that we pick up tensorflow/tensorflow@b898082, where a new experimental shape inference API is introduced.

@bgogul bgogul force-pushed the move_tensorflow branch from d3d8c9b to bd7d702 Compare July 25, 2019 07:54
@bgogul
Copy link
Contributor Author

bgogul commented Jul 25, 2019

@swift-ci please clean test tensorflow

@bgogul bgogul force-pushed the move_tensorflow branch from bd7d702 to e2bace4 Compare July 25, 2019 16:50
@bgogul
Copy link
Contributor Author

bgogul commented Jul 25, 2019

@swift-ci please clean test tensorflow

@bgogul bgogul force-pushed the move_tensorflow branch from e2bace4 to a0105d8 Compare July 25, 2019 17:51
@bgogul
Copy link
Contributor Author

bgogul commented Jul 25, 2019

@swift-ci please clean test tensorflow

@bgogul
Copy link
Contributor Author

bgogul commented Jul 26, 2019

@swift-ci please clean test tensorflow gpu

@bgogul
Copy link
Contributor Author

bgogul commented Jul 26, 2019

@swift-ci please clean test tensorflow-gpu

@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow

1 similar comment
@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow

@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow-gpu

2 similar comments
@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow-gpu

@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow-gpu

@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow gpu

@bgogul bgogul force-pushed the move_tensorflow branch from a0105d8 to 87c1dbc Compare July 29, 2019 23:56
@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow gpu

@bgogul
Copy link
Contributor Author

bgogul commented Jul 29, 2019

@swift-ci please clean test tensorflow

@bgogul
Copy link
Contributor Author

bgogul commented Jul 30, 2019

@swift-ci please clean test tensorflow gpu

1 similar comment
@bgogul
Copy link
Contributor Author

bgogul commented Jul 30, 2019

@swift-ci please clean test tensorflow gpu

@bgogul bgogul requested a review from pschuh July 30, 2019 06:06
@bgogul
Copy link
Contributor Author

bgogul commented Jul 30, 2019

@swift-ci please clean test tensorflow linux

@bgogul
Copy link
Contributor Author

bgogul commented Jul 30, 2019

@swift-ci please clean test tensorflow

@bgogul
Copy link
Contributor Author

bgogul commented Jul 30, 2019

The GPU tests are failing because the nccl files in ~/.cache/bazel is not compatible with the tensorflow commit hash. This should build if the ~/.cache/bazel is completely removed. See the successful run here:
https://ci-external.swift.org/view/Pull%20Request/job/swift-PR-TensorFlow-Linux-gpu/1853/

Merging this in now.

@bgogul bgogul merged commit 19c9e5f into swiftlang:tensorflow Jul 30, 2019
@bgogul bgogul deleted the move_tensorflow branch July 30, 2019 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pschuh pschuh pschuh approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bgogul @pschuh