Skip to content

Conversation

@mansnils
Copy link
Collaborator

@mansnils mansnils commented Sep 9, 2025

Enable VKML op unit test for Arm backend.

cc @digantdesai @freddan80 @per @zingo @oscarandersson8218

Change-Id: I58fa82ba8c55103c934a1a0b309d6e7c1327d35e
@mansnils mansnils added partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm ciflow/trunk labels Sep 9, 2025
@pytorch-bot
Copy link

pytorch-bot bot commented Sep 9, 2025

🔗 Helpful Links

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

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

⏳ 1 Pending, 1 Unrelated Failure

As of commit f36640a with merge base 8c48a76 (image):

FLAKY - The following job failed but was likely due to flakiness present on trunk:

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

@meta-cla meta-cla bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 9, 2025
@zingo
Copy link
Collaborator

zingo commented Sep 9, 2025

Ouch, all jobs timedout installing/building the deps :(
Until we get precompiled deps this might need to be pre-build in the docker.
Maybe the Vulkan backend already have a solution to copy/clone?

@mansnils mansnils added the release notes: arm Changes to the ARM backend delegate label Oct 24, 2025
@mansnils
Copy link
Collaborator Author

mansnils commented Nov 7, 2025

Missing building executor runner for VKML - will add it.

@mansnils mansnils requested a review from cccclai as a code owner November 7, 2025 12:56
@mansnils
Copy link
Collaborator Author

mansnils commented Nov 7, 2025

Missing building executor runner for VKML - will add it.

Done

@mansnils
Copy link
Collaborator Author

mansnils commented Nov 8, 2025

Let's see if [trunk / test-arm-backend (test_pytest_ops_vkml) pass with linux.g5.4xlarge.nvidia.gpu instead of linux.2xlarge.memory runner just as a quick test.

@mansnils
Copy link
Collaborator Author

The docker update probably need to merge into a separate PR first.

@mansnils
Copy link
Collaborator Author

The docker update probably need to merge into a separate PR first.

#15812

@mansnils mansnils merged commit 4178419 into pytorch:main Nov 14, 2025
723 of 731 checks passed
@zingo
Copy link
Collaborator

zingo commented Nov 15, 2025

After this was merged one Arm job times out all the time :(
See trunk / test-arm-backend (test_pytest_ops_ethosu_fvp) on
https://hud.pytorch.org/hud/pytorch/executorch/main/1?per_page=50&name_filter=arm-backend

zingo added a commit that referenced this pull request Nov 15, 2025
@zingo
Copy link
Collaborator

zingo commented Nov 15, 2025

Creating a revert PR #15841

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/trunk CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. partner: arm For backend delegation, kernels, demo, etc. from the 3rd-party partner, Arm release notes: arm Changes to the ARM backend delegate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants