Skip to content

Conversation

seemethere
Copy link
Member

@seemethere seemethere commented Oct 16, 2020

ROCM builds were being picked up for CUDA 10.2 builds since they do not
specify a version suffix. We can utilize the CU_VERSION to be specific
about which index to actually pull from.

Logs of what happened during the release: https://app.circleci.com/pipelines/github/pytorch/vision/4454/workflows/efcd7fa9-7f94-46ef-bee3-a70cefd2dcd2/jobs/266058

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

ROCM builds were being picked up for CUDA 10.2 builds since they do not
specify a version suffix. We can utilize the CU_VERSION to be specific
about which index to actually pull from.

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
@seemethere seemethere requested review from a team and fmassa October 16, 2020 16:55
@seemethere seemethere changed the title .cirlceci: Be specific about CU_VERSION for pytorch [release/0.8] .cirlceci: Be specific about CU_VERSION for pytorch Oct 16, 2020
Copy link
Member

@fmassa fmassa left a comment

Choose a reason for hiding this comment

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

LGTM.

Will we be providing ROCm nightlies as well? If yes we might want to send a similar fix for the master branch

@seemethere
Copy link
Member Author

LGTM.

Will we be providing ROCm nightlies as well? If yes we might want to send a similar fix for the master branch

We actually do provide ROCM binaries already for nightlies, so I'm not entirely sure why this didn't present itself in the nightlies, but in any case this'll get us unblocked for today.

#2827 is the version of this PR for master

@fmassa
Copy link
Member

fmassa commented Oct 16, 2020

Test failures only apply to the new cmake_* jobs, which might be bogus for a release branch and are only used for testing a few things. I wouldn't bother about it if other tests pass

@seemethere
Copy link
Member Author

Checks have passed the area they failed previously so going to go ahead and merge

@seemethere seemethere merged commit 1f629a5 into pytorch:release/0.8.0 Oct 16, 2020
@seemethere seemethere deleted the be_specific_about_cu_version branch October 16, 2020 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants