Skip to content

Re-enable rocm tests that have been fixed in rocm 1.8.2 #9862

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

Closed
wants to merge 1 commit into from

Conversation

bddppq
Copy link
Contributor

@bddppq bddppq commented Jul 26, 2018

@petrex
Copy link
Contributor

petrex commented Jul 26, 2018

@bddppq need to update the bare metal as well. let's find a good time to do that.

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bddppq has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@bddppq bddppq closed this Jul 26, 2018
@bddppq bddppq reopened this Jul 26, 2018
@bddppq
Copy link
Contributor Author

bddppq commented Jul 26, 2018

@petrex these tests seem to be passing fine, so I'm going to land this now.
Let's coordinate offline how to update the rocm packages on CI boxes.

@bddppq bddppq deleted the rocm-tests branch July 26, 2018 17:58
jramseyer pushed a commit to jramseyer/pytorch that referenced this pull request Jul 30, 2018
Summary:
petrex
Pull Request resolved: pytorch#9862

Differential Revision: D9012520

Pulled By: bddppq

fbshipit-source-id: cdcc184e23befa8dbd1bc44d59bd25766aac33d0
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
petrex
Pull Request resolved: pytorch#9862

Differential Revision: D9012520

Pulled By: bddppq

fbshipit-source-id: cdcc184e23befa8dbd1bc44d59bd25766aac33d0
@ezyang ezyang added the merged label Jun 26, 2019
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

Successfully merging this pull request may close these issues.

5 participants