Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add OSS GPU tests #231

Closed
wants to merge 7 commits into from
Closed

Add OSS GPU tests #231

wants to merge 7 commits into from

Conversation

PaliC
Copy link
Contributor

@PaliC PaliC commented Oct 26, 2022

Stack from ghstack (oldest at bottom):

Adds GPU tests to OSS CI to ensure we don't break cuda related things by accident.

I had to make some updates to our dockerfile as the cuda dependencies for nightlies changed.

Differential Revision: D40727173

[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: bf0b9661ac3569e0e9b5ffd103575a37b8148d97
Pull Request resolved: #231
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 26, 2022
[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: d38a0bf5a9b29b000f60396d9bfb6b6df21229e0
Pull Request resolved: #231
[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: 9461ee2cf87349ecafc403e68ad5dc92a71110af
Pull Request resolved: #231
[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: c427425d8c7392034dca50c55a8c58205ef0ef10
Pull Request resolved: #231
[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: 8e53ef2adcca4619553d55ad2b493122ca05e9a9
Pull Request resolved: #231
@PaliC
Copy link
Contributor Author

PaliC commented Oct 26, 2022

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

Adds GPU tests to OSS CI to ensure we don't break cuda related things by accident. 

I had to make some updates to our dockerfile as the cuda dependencies for nightlies changed.

Differential Revision: [D40727173](https://our.internmc.facebook.com/intern/diff/D40727173)

[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: 8583285fe11983d6686c8551ba8eb87a154143b4
Pull Request resolved: #231
@facebook-github-bot
Copy link
Contributor

@PaliC has updated the pull request. You must reimport the pull request before landing.

@PaliC
Copy link
Contributor Author

PaliC commented Oct 26, 2022

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

Adds GPU tests to OSS CI to ensure we don't break cuda related things by accident. 

I had to make some updates to our dockerfile as the cuda dependencies for nightlies changed.

Differential Revision: [D40727173](https://our.internmc.facebook.com/intern/diff/D40727173)

[ghstack-poisoned]
PaliC added a commit that referenced this pull request Oct 26, 2022
ghstack-source-id: 093cf502a4de22311dcbe770d88f58120d48bb29
Pull Request resolved: #231
@facebook-github-bot
Copy link
Contributor

@PaliC has updated the pull request. You must reimport the pull request before landing.

@PaliC
Copy link
Contributor Author

PaliC commented Oct 26, 2022

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

@facebook-github-bot facebook-github-bot deleted the gh/PaliC/75/head branch October 30, 2022 14:19
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants