Skip to content

Conversation

@xuzhao9
Copy link
Contributor

@xuzhao9 xuzhao9 commented Aug 30, 2022

CUDA 11.3 is being deprecated, and we are using environment variables to control the CUDA version instead of changing the link target of /usr/local/cuda.

This has many benefits:

  • switching cuda version no longer needs sudo permission
  • support switching cuda versions at both runtime and compile time

This PR also creates a cuda_utils.py to place all CUDA version related code.

@facebook-github-bot
Copy link
Contributor

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

@xuzhao9 xuzhao9 changed the title Setup CUDA environment when running bisection Setup CUDA environment when running bisection and abtest Aug 30, 2022
@xuzhao9 xuzhao9 requested a review from davidberard98 August 30, 2022 19:24
@facebook-github-bot
Copy link
Contributor

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

Copy link
Contributor

@davidberard98 davidberard98 left a comment

Choose a reason for hiding this comment

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

Thanks!

@xuzhao9 xuzhao9 deleted the xz9/setup-cuda-compile branch August 30, 2022 20:54
pytorchmergebot pushed a commit to pytorch/pytorch that referenced this pull request Aug 30, 2022
Upgrade CUDA version of torchbench as we are moving away from CUDA 11.3
This PR needs to land together with pytorch/benchmark#1141

RUN_TORCHBENCH: nvfuser
TORCHBENCH_BRANCH: xz9/setup-cuda-compile

Pull Request resolved: #84248
Approved by: https://github.com/erichan1, https://github.com/davidberard98
facebook-github-bot pushed a commit to pytorch/pytorch that referenced this pull request Sep 1, 2022
Summary:
Upgrade CUDA version of torchbench as we are moving away from CUDA 11.3
This PR needs to land together with pytorch/benchmark#1141

RUN_TORCHBENCH: nvfuser
TORCHBENCH_BRANCH: xz9/setup-cuda-compile

Pull Request resolved: #84248
Approved by: https://github.com/erichan1, https://github.com/davidberard98

Test Plan: contbuild & OSS CI, see https://hud.pytorch.org/commit/pytorch/pytorch/e014bd8e4ef65377374640310cbafbccbcd0f5f7

Reviewed By: mehtanirav

Differential Revision: D39171934

Pulled By: xuzhao9

fbshipit-source-id: 6c627b902b5d425f2b4dfd8666896cd4a69e1d93
xuzhao9 added a commit that referenced this pull request Sep 6, 2022
Summary:
CUDA 11.3 is being deprecated, and we are using environment variables to control the CUDA version instead of changing the link target of `/usr/local/cuda`.

This has many benefits:
- switching cuda version no longer needs sudo permission
- support switching cuda versions at both runtime and compile time

This PR also creates a `cuda_utils.py` to place all CUDA version related code.

Pull Request resolved: #1141

Reviewed By: erichan1, davidberard98

Differential Revision: D39139827

Pulled By: xuzhao9

fbshipit-source-id: 6c305ab749c8cfc0d5bc8f15a86dc4a0a6d04a96
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.

3 participants