Skip to content

ci: Add cu102 to CI and packaging, remove cu100 #1980

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

Merged
merged 4 commits into from
Mar 19, 2020

Conversation

seemethere
Copy link
Member

Adds cu102 to our testing matrix as well as change the default manywheel
version to be cu102.

Signed-off-by: Eli Uriegas eliuriegas@fb.com

@seemethere seemethere requested a review from fmassa March 13, 2020 18:41
@seemethere seemethere force-pushed the add_cu102 branch 6 times, most recently from afd67a3 to a34fc6d Compare March 16, 2020 19:03
@seemethere seemethere requested a review from peterjc123 March 16, 2020 20:17
@seemethere
Copy link
Member Author

@peterjc123 Do we not have cuda 10.2 on the windows executors?

@peterjc123
Copy link
Contributor

@seemethere No, CUDA 10.2 is not in the circleci image. You'll have to install it manually.

Adds cu102 to our testing matrix as well as change the default manywheel
version to be cu102.

Also adds a function to get manylinux images instead of it being a
conditional block.

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
We removed the build for cu100 in upstream pytorch as well since we have
begun to support cu102

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
@seemethere seemethere changed the title ci: Add cu102 to CI and packaging ci: Add cu102 to CI and packaging, remove cu100 Mar 18, 2020
@fmassa fmassa merged commit d45a77d into pytorch:master Mar 19, 2020
@fmassa
Copy link
Member

fmassa commented Mar 19, 2020

Thanks a lot @seemethere !

fmassa added a commit to fmassa/vision-1 that referenced this pull request Mar 20, 2020
* ci: Add cu102 to CI and packaging

Adds cu102 to our testing matrix as well as change the default manywheel
version to be cu102.

Also adds a function to get manylinux images instead of it being a
conditional block.

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

* .circleci: Remove cu100, no longer built

We removed the build for cu100 in upstream pytorch as well since we have
begun to support cu102

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>

* Fix lint

* More lint fixes

Co-authored-by: Francisco Massa <fvsmassa@gmail.com>
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.

3 participants