Skip to content

[Caffe2] Pin AMD gpu id in Caffe2 CI #10144

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
Closed

Conversation

bddppq
Copy link
Contributor

@bddppq bddppq commented Aug 1, 2018

@bddppq bddppq requested review from ezyang, Jorghi12 and houseroad August 1, 2018 22:23
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.

Copy link
Member

@houseroad houseroad left a comment

Choose a reason for hiding this comment

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

I am wondering if we can launch 4 CI jobs on the same box?

@bddppq
Copy link
Contributor Author

bddppq commented Aug 1, 2018

@houseroad I'm going to do 2 for now, round robin on build number is actually not very robust, since builds can be canceled (not very rare).

@petrex
Copy link
Contributor

petrex commented Aug 2, 2018

This is why I like to work with you!@bddppq

@bddppq bddppq deleted the pin-hip-gpu branch August 2, 2018 00:40
goodlux pushed a commit to goodlux/pytorch that referenced this pull request Aug 15, 2018
Summary:
petrex
Pull Request resolved: pytorch#10144

Differential Revision: D9125707

Pulled By: bddppq

fbshipit-source-id: 8ef8f3da6ceb1855f28fc24be621b9b4854ff7f9
@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