Skip to content

Fix torchaudio build when TORCH_CUDA_ARCH_LIST is not set#161084

Closed
huydhn wants to merge 3 commits intopytorch:mainfrom
huydhn:fix-torchaudio-build-cuda-arch-not-set
Closed

Fix torchaudio build when TORCH_CUDA_ARCH_LIST is not set#161084
huydhn wants to merge 3 commits intopytorch:mainfrom
huydhn:fix-torchaudio-build-cuda-arch-not-set

Conversation

@huydhn
Copy link
Contributor

@huydhn huydhn commented Aug 20, 2025

Fixes #160988. The root cause can be found in the same issue. This fix ensures that when reuse old wheel is on and torchaudio wheel is not there, the inductor test job can still rebuild the wheel it needs

Signed-off-by: Huy Do <huydhn@gmail.com>
@huydhn huydhn requested review from malfet and zou3519 August 20, 2025 19:53
@huydhn huydhn requested a review from a team as a code owner August 20, 2025 19:53
@pytorch-bot
Copy link

pytorch-bot bot commented Aug 20, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 923e661 with merge base 4e90441 (image):
💚 Looks good so far! There are no failures yet. 💚

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

@huydhn huydhn added the topic: not user facing topic category label Aug 20, 2025
@malfet
Copy link
Contributor

malfet commented Aug 20, 2025

Looks ok, but do you mind leaving a comment that references the issue? And may be explain a path forward. I.e. why torchaudio fails to autodetect GPU architecture itself?

Copy link
Contributor

@zou3519 zou3519 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

Signed-off-by: Huy Do <huydhn@gmail.com>
@huydhn
Copy link
Contributor Author

huydhn commented Aug 21, 2025

@pytorchbot merge -f 'CI only tweak, no need to run trunk'

@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
Copy link
Contributor Author

huydhn commented Aug 21, 2025

@pytorchbot revert -m 'My mistake for not checking for nvidia-smi availability' -c weird

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a revert job. Check the current status here.
Questions? Feedback? Please reach out to the PyTorch DevX Team

pytorchmergebot added a commit that referenced this pull request Aug 21, 2025
…61084)"

This reverts commit cfdaaaa.

Reverted #161084 on behalf of https://github.com/huydhn due to My mistake in not checking for nvidia-smi availability ([comment](#161084 (comment)))
@pytorchmergebot
Copy link
Collaborator

@huydhn your PR has been successfully reverted.

@pytorchmergebot pytorchmergebot added Reverted ci-no-td Do not run TD on this PR labels Aug 21, 2025
Signed-off-by: Huy Do <huydhn@gmail.com>
@huydhn
Copy link
Contributor Author

huydhn commented Aug 21, 2025

@pytorchbot merge -f 'No need to run trunk, inductor jobs are ok'

@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

markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
…1084)

Fixes pytorch#160988.  The root cause can be found in the same issue.  This fix ensures that when reuse old wheel is on and `torchaudio` wheel is not there, the inductor test job can still rebuild the wheel it needs
Pull Request resolved: pytorch#161084
Approved by: https://github.com/malfet, https://github.com/zou3519
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
…torch#161084)"

This reverts commit cfdaaaa.

Reverted pytorch#161084 on behalf of https://github.com/huydhn due to My mistake in not checking for nvidia-smi availability ([comment](pytorch#161084 (comment)))
markc-614 pushed a commit to markc-614/pytorch that referenced this pull request Sep 17, 2025
…1084)

Fixes pytorch#160988.  The root cause can be found in the same issue.  This fix ensures that when reuse old wheel is on and `torchaudio` wheel is not there, the inductor test job can still rebuild the wheel it needs
Pull Request resolved: pytorch#161084
Approved by: https://github.com/malfet, https://github.com/zou3519
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.

Inductor torchbench CI tests are failing to build torchaudio

4 participants