pytorch / executorch

On-device AI across mobile, embedded and edge for PyTorch
https://pytorch.org/executorch/
Other
1.35k stars 218 forks source link

Enable Arm unit test when running pull jobs #3371

Closed zingo closed 1 week ago

zingo commented 1 month ago

Trigger and run backend arm tests as par of github ci

pytorch-bot[bot] commented 1 month ago

:link: Helpful Links

:test_tube: See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/3371

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

:white_check_mark: No Failures

As of commit feaa5a641ff95d240ce9ae83eb40bce26a91c353 with merge base 69f4a2fd09c079eb5af2fa09912bfce6d6ae9a23 (image): :green_heart: Looks good so far! There are no failures yet. :green_heart:

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

zingo commented 1 month ago

There are some different way to do this and maybe we could also limit scope of testing to only arm backend test for this job to make it faster, please feel free to comment suggest on your thoughts. :)

pytorch-bot[bot] commented 1 month ago

Please seek CI approval before scheduling CIFlow labels

zingo commented 1 month ago

@pytorchbot label ciflow/trunk

pytorch-bot[bot] commented 1 month ago

Can't add following labels to PR: ciflow/trunk. Please ping one of the reviewers for help.

zingo commented 1 month ago

@pytorchbot label ciflow/trunk

pytorch-bot[bot] commented 1 month ago

Can't add following labels to PR: ciflow/trunk. Please ping one of the reviewers for help.

pytorch-bot[bot] commented 1 month ago

Please seek CI approval before scheduling CIFlow labels

robell commented 1 month ago

@pytorchbot label ciflow/trunk

pytorch-bot[bot] commented 1 month ago

Can't add following labels to PR: ciflow/trunk. Please ping one of the reviewers for help.

zingo commented 3 weeks ago

@digantdesai @guangy10 anything more you want me to do in this pull request?

digantdesai commented 3 weeks ago

Nothing from my side, we are discussing internally but hopefully should resolve today.

zingo commented 2 weeks ago

Anything we can change/update in the patch to improve the chance of a fruitful outcome? 😊

Is the time a problem at all?

zingo commented 1 week ago

@pytorchbot label ciflow/trunk

facebook-github-bot commented 1 week ago

@digantdesai has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

digantdesai commented 1 week ago

Trying to merge now.

facebook-github-bot commented 1 week ago

@digantdesai merged this pull request in pytorch/executorch@610eb4fea394a2e76b33fc34d6fab87a48251d8a.