KEMBAR78
Enable py3.13 wheels for ROCm by jithunnair-amd · Pull Request #142294 · pytorch/pytorch · GitHub
Skip to content

Conversation

jithunnair-amd
Copy link
Collaborator

@jithunnair-amd jithunnair-amd commented Dec 7, 2024

@pytorch-bot
Copy link

pytorch-bot bot commented Dec 7, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/142294

Note: Links to docs will display an error until the docs builds have been completed.

❌ 5 New Failures

As of commit 985aa6c with merge base c863227 (image):

NEW FAILURES - The following jobs have failed:

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@pytorch-bot pytorch-bot bot added ciflow/rocm Trigger "default" config CI on ROCm module: rocm AMD GPU support for Pytorch topic: not user facing topic category labels Dec 7, 2024
@jithunnair-amd jithunnair-amd added the ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR label Dec 7, 2024
@jithunnair-amd jithunnair-amd marked this pull request as ready for review December 9, 2024 01:32
@jithunnair-amd jithunnair-amd requested a review from a team as a code owner December 9, 2024 01:32
@jithunnair-amd
Copy link
Collaborator Author

jithunnair-amd commented Dec 9, 2024

@atalman ROCm py3.13 binary wheel jobs passed. Please approve and merge. I would like to Cherry pick this for PyTorch 2.6 so that we can enable 3.13 wheels for ROCm for the latest release.

@janeyx99 janeyx99 added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Dec 9, 2024
Copy link
Contributor

@huydhn huydhn left a comment

Choose a reason for hiding this comment

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

LGTM!

@huydhn
Copy link
Contributor

huydhn commented Dec 10, 2024

@pytorchbot merge -f 'No need to run trunk jobs, ROCm build for 3.13 and 3.13t have passed'

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged immediately since you used the force (-f) flag, bypassing any CI checks (ETA: 1-5 minutes). Please use -f as last resort and instead consider -i/--ignore-current to continue the merge ignoring current failures. This will allow currently pending tests to finish and report signal before the merge.

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@huydhn huydhn removed ciflow/binaries_wheel Trigger binary build and upload jobs for wheel on the PR ciflow/rocm Trigger "default" config CI on ROCm labels Dec 10, 2024
pytorchmergebot pushed a commit that referenced this pull request Dec 10, 2024
Fixes #142485

The workflow check lint job timed out in trunk, i.e. https://github.com/pytorch/pytorch/actions/runs/12261226178/job/34207762939, and here was what happened:

1. #142294 landed yesterday to build ROCm on 3.13, but the PR had a landrace with #142282 in the generated workflow file
2. The trunk lint check caught that in https://github.com/pytorch/pytorch/blob/main/.github/scripts/report_git_status.sh#L2
3. However, the script also attempted to print the difference with `git diff .github/workflows`.  This command was the one that stuck because `git diff` uses page by default and requires a prompt to display the next page ¯\_(ツ)_/¯

It took so long to debug this because a timeout Nova GHA doesn't print any progress.  I'll create an issue for this.

Bonus:

I also fix the broken print from test tool lint job that confuses GitHub https://github.com/pytorch/pytorch/actions/runs/12261226178 with an annotation failure `Credentials could not be loaded, please check your action inputs`

Pull Request resolved: #142476
Approved by: https://github.com/wdvr
@atalman atalman mentioned this pull request Dec 12, 2024
37 tasks
atalman added a commit to pytorch/test-infra that referenced this pull request Dec 17, 2024
atalman added a commit to atalman/test-infra that referenced this pull request Dec 17, 2024
@github-actions github-actions bot deleted the enable_py3.13_wheels_for_rocm branch January 18, 2025 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Merged module: rocm AMD GPU support for Pytorch open source topic: not user facing topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants