Skip to content

Add torchao to install_inductor_benchmark_deps cleanup stage #160191

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

Conversation

robert-hardwick
Copy link
Collaborator

@robert-hardwick robert-hardwick commented Aug 8, 2025

It looks like torcho was missed from the cleanup during torchbench setup.

Stack from ghstack (oldest at bottom):

Fixes #160188

cc @seemethere @malfet @pytorch/pytorch-dev-infra @snadampal @milpuz01 @aditew01 @nikhil-arm @fadara01

[ghstack-poisoned]
Copy link

pytorch-bot bot commented Aug 8, 2025

🔗 Helpful Links

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

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

❌ 15 New Failures

As of commit 2f1347d with merge base 556e2a7 (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 the topic: not user facing topic category label Aug 8, 2025
robert-hardwick added a commit that referenced this pull request Aug 8, 2025
@robert-hardwick
Copy link
Collaborator Author

@pytorchbot label "ciflow/inductor-perf-nightly-aarch64"

Copy link

pytorch-bot bot commented Aug 8, 2025

Didn't find following labels among repository labels: ciflow/inductor-perf-nightly-aarch64

@robert-hardwick
Copy link
Collaborator Author

@pytorchbot label "topic: not user facing"

@robert-hardwick
Copy link
Collaborator Author

@pytorchbot label "ciflow/inductor-perf-test-nightly-aarch64"

Copy link

pytorch-bot bot commented Aug 8, 2025

Didn't find following labels among repository labels: ciflow/inductor-perf-test-nightly-aarch64

@robert-hardwick robert-hardwick added module: ci Related to continuous integration module: arm Related to ARM architectures builds of PyTorch. Includes Apple M1 labels Aug 8, 2025
@robert-hardwick
Copy link
Collaborator Author

Looks like it has fixed the issue - > https://github.com/pytorch/pytorch/actions/runs/16833390878/job/47686603854

@@ -48,4 +48,4 @@ install_huggingface
install_timm

# Clean up
conda_run pip uninstall -y torch torchvision torchaudio triton
conda_run pip uninstall -y torch torchvision torchaudio triton torchao
Copy link
Contributor

Choose a reason for hiding this comment

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

I just realize that there is one more permission error bug in this script. After installing TorchBench on line 37, the script runs under root as part of the Docker build, so another chown -R jenkins /opt/conda is needed similar to https://github.com/pytorch/pytorch/blob/main/.ci/docker/common/install_conda.sh#L21. This will fix the remaining error in this script

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.

Thank you for the fix! Uninstalling torchao is the right thing to do here. There is another permission bug from my comment, but feel free to fix it here together with this change, or I could submit another PR to fix it after this lands. It's up to you

@huydhn
Copy link
Contributor

huydhn commented Aug 8, 2025

@pytorchbot merge -f 'Let land this to unblock #159291, I will submit a fix for the rest of these failures in a separate PR'

@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

hinriksnaer pushed a commit to hinriksnaer/pytorch that referenced this pull request Aug 8, 2025
…#160191)

It looks like `torcho` was missed from the cleanup during torchbench setup.

Fixes pytorch#160188

Pull Request resolved: pytorch#160191
Approved by: https://github.com/huydhn
@robert-hardwick
Copy link
Collaborator Author

Thank you for the fix! Uninstalling torchao is the right thing to do here. There is another permission bug from my comment, but feel free to fix it here together with this change, or I could submit another PR to fix it after this lands. It's up to you

Thanks for the approval and for merging @huydhn . I would have fixed the permissions in this PR, but we have a process to follow and it was Fri PM in UK, so I wouldn't be able to make the change until Monday, so better to merge as-is 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arm priority Merged module: arm Related to ARM architectures builds of PyTorch. Includes Apple M1 module: ci Related to continuous integration open source topic: bug fixes topic category topic: not user facing topic category
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants