Skip to content

[Backend Tester] Add FACTO operator test skeleton #11953

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

GregoryComer
Copy link
Member

@GregoryComer GregoryComer commented Jun 25, 2025

Add initial skeleton for running Facto operator tests. This is only set up for XNNPACK in this commit (other delegates are later in the stack). It also relies on a manual install of Facto. I'm currently intentionally not running this in CI, due to a combination of finding a number of crashes and due to a high volume of tests. This will be addressed further up the stack. Instructions for running the tests locally are included at the top of test_facto.py.

[ghstack-poisoned]
@GregoryComer
Copy link
Member Author

GregoryComer commented Jun 25, 2025

Copy link

pytorch-bot bot commented Jun 25, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit 7bdd922 with merge base 91c9ffa (image):
💚 Looks good so far! There are no failures yet. 💚

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

GregoryComer added a commit that referenced this pull request Jun 25, 2025
ghstack-source-id: a082c5e
ghstack-comment-id: 3003288787
Pull-Request: #11953
@facebook-github-bot facebook-github-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 Jun 25, 2025
@GregoryComer GregoryComer added the release notes: none Do not include this in the release notes label Jun 25, 2025
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Jun 25, 2025
ghstack-source-id: 101d2b7
ghstack-comment-id: 3003288787
Pull-Request: #11953
[ghstack-poisoned]
GregoryComer added a commit that referenced this pull request Jun 25, 2025
ghstack-source-id: 7298522
ghstack-comment-id: 3003288787
Pull-Request: #11953
[ghstack-poisoned]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. release notes: none Do not include this in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants