Skip to content

Outdated ROCm link in https://pytorch.org/get-started/locally/ #1673

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

Closed
Utorque opened this issue Jul 12, 2024 · 1 comment · Fixed by #1675
Closed

Outdated ROCm link in https://pytorch.org/get-started/locally/ #1673

Utorque opened this issue Jul 12, 2024 · 1 comment · Fixed by #1675
Assignees

Comments

@Utorque
Copy link

Utorque commented Jul 12, 2024

🕋 Website

On the start locally page, when choosing linux, the documentation link to ROCm is outdated.

To Reproduce

When choosing "Linux" in https://pytorch.org/get-started/locally/ :
image
Just below, in the "Installing on Linux" paragraph, the ROCm link :
image
It leads to this page : https://docs.amd.com/
image

Expected behavior

It should however redirect directly to https://rocm.docs.amd.com/en/latest/
image

Additional context

Note that it would be interesting to redirect to the correct version of ROCm instead of the latest as suggested in Expected behavior. For example, it could be https://rocm.docs.amd.com/en/docs-6.0.2/ for the stable build and https://rocm.docs.amd.com/en/docs-6.1.2/ for the nightly (not sure about the exact versions !).

@Utorque Utorque changed the title Wrong ROCm link in https://pytorch.org/get-started/locally/ Outdated ROCm link in https://pytorch.org/get-started/locally/ Jul 12, 2024
@Utorque
Copy link
Author

Utorque commented Jul 12, 2024

It's also the case here :
https://pytorch.org/get-started/locally/#linux-prerequisites-2
Both ROCm and ROCm-enabled GPU links are outdated. ROCm link should be the same as above. The GPU link should be this one : https://rocm.docs.amd.com/projects/install-on-linux/en/latest/reference/system-requirements.html

@svekars svekars self-assigned this Jul 15, 2024
malfet pushed a commit that referenced this issue Jul 16, 2024
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 a pull request may close this issue.

2 participants