Skip to content
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

docs(ansible/cuda): fix outdated cuda version in README #2986

Merged
merged 1 commit into from
Nov 1, 2022

Conversation

tleyden
Copy link
Contributor

@tleyden tleyden commented Oct 31, 2022

Description

Update the recommended version of CUDA to install from 11-4 -> 11-6 based on this discussion.

Note that this value matches the version in the ansible env variable.

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

Signed-off-by: Traun Leyden <traun.leyden@gmail.com>
@tleyden tleyden changed the title Update cuda version Fix outdated cuda version in README Oct 31, 2022
@tleyden tleyden changed the title Fix outdated cuda version in README Fix: outdated cuda version in README Oct 31, 2022
@tleyden tleyden changed the title Fix: outdated cuda version in README fix(ansible): outdated cuda version in README Oct 31, 2022
Copy link
Contributor

@shmpwk shmpwk left a comment

Choose a reason for hiding this comment

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

Thank you for fixing!

@kenji-miyake kenji-miyake changed the title fix(ansible): outdated cuda version in README docs(ansible/cuda): fix outdated cuda version in README Nov 1, 2022
@kenji-miyake kenji-miyake merged commit 62399f5 into autowarefoundation:main Nov 1, 2022
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