Skip to content

Arm64 workflow enablement #8947

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

Merged
merged 27 commits into from
Jun 26, 2025
Merged

Conversation

alinpahontu2912
Copy link
Collaborator

Added workflow for arm64 build of torchvision

Copy link

pytorch-bot bot commented Mar 3, 2025

🔗 Helpful Links

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

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

❌ 2 New Failures, 2 Pending

As of commit f842b49 with merge base 0721867 (image):

NEW FAILURES - The following jobs have failed:

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

@alinpahontu2912 alinpahontu2912 marked this pull request as ready for review March 5, 2025 11:37
@NicolasHug
Copy link
Member

NicolasHug commented May 29, 2025

Hi @alinpahontu2912, I see this is related to #9080 which I'm also a bit confused about.

Can you please share some context about this? As far as I know pytorch currently doesn't support Arm64 Windows. @seemethere @malfet , are we actively trying to support arm64 + Windows? What would be the support model for this platform?

@iremyux
Copy link
Collaborator

iremyux commented Jun 4, 2025

Hey @NicolasHug , Pytorch supports Windows Arm64. Nightly & 2.7.0 wheels are available, only not on PyPi yet. @alinpahontu2912 is currently working on Windows Arm64 support for torchvision and torchaudio. This PR, #9080 and pytorch/test-infra#6352 are all a part of this enablement.

@alinpahontu2912
Copy link
Collaborator Author

Hey @seemethere @malfet can you also take a look at these changes?

@NicolasHug
Copy link
Member

@alinpahontu2912 @iremyux @aalina23 Thank you for the PR.

Would it be possible to handle these build jobs in a separate repository that you would own, instead of within the torchvision repo?

This would help us manage the fixed maintenance cost associated with new CI jobs. Even if we aren't directly maintaining these jobs, we still have to merge your maintenance PRs and deal with potential failures, which can add noise to our CI. By using a separate repo, you could work more swiftly without being held up by our reviews, and we could avoid the fixed mainenance cost on our side.

Maybe @seemethere can advise on the technical feasibility of this idea?

@alinpahontu2912
Copy link
Collaborator Author

Hey @NicolasHug with the recent test-infra PR that was merged, a build of the arm64 Windows package is already triggered by the new test workflow whenever relevant files are changed. Given this, I think splitting the build logic into a separate repo wouldn’t reduce CI activity and would instead introduce unnecessary complexity and split ownership. We think keeping everything within the current setup is more aligned with how the CI is already structured. Happy to discuss this further, though

@malfet malfet merged commit 6aee5ed into pytorch:main Jun 26, 2025
60 of 64 checks passed
Copy link

Hey @malfet!

You merged this PR, but no labels were added.
The list of valid labels is available at https://github.com/pytorch/vision/blob/main/.github/process_commit.py

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.

6 participants