[xnnpack] Fix pyre issues for deconv #7656

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.

Sign up for GitHub

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

Jump to bottom

Closed

digantdesai wants to merge 2 commits into main from deconv_pyre

Contributor

digantdesai commented Jan 14, 2025

Summary

Fix pyre Issues for an OSS PR

digantdesai added the module: xnnpack label

digantdesai requested a review from mcr229

January 14, 2025 19:42

digantdesai self-assigned this

pytorch-bot bot commented Jan 14, 2025 •

edited

Loading

🔗 Helpful Links

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

📄 Preview Python docs built from this PR

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

❌ 1 New Failure, 3 Pending

As of commit ba8e2c7 with merge base ee1d7c3 ():

NEW FAILURE - The following job has failed:

pull / test-phi-3-mini-runner-linux / linux-job (gh)
RuntimeError: Command docker exec -t 508d607976a018a8e23a00af6b35fe90117dd56314d1f9a5739e383908ff358d /exec failed with exit code 2

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

facebook-github-bot added the CLA Signed label

Contributor

facebook-github-bot commented Jan 14, 2025

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

digantdesai added the topic: not user facing label

mcr229 approved these changes

View reviewed changes

digantdesai force-pushed the deconv_pyre branch from f0bec69 to b58a171 Compare

January 15, 2025 20:21

Contributor

facebook-github-bot commented Jan 15, 2025

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

1 similar comment

Contributor

facebook-github-bot commented Jan 16, 2025

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

digantdesai force-pushed the deconv_pyre branch from b58a171 to abe80ef Compare

January 16, 2025 04:27

Contributor

facebook-github-bot commented Jan 16, 2025

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

digantdesai force-pushed the deconv_pyre branch 2 times, most recently from 08f455c to 4f62baf Compare

January 17, 2025 03:06

Contributor

facebook-github-bot commented Jan 17, 2025

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

1 similar comment

Contributor

facebook-github-bot commented Jan 17, 2025

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


          [xnnpack] Fix pyre issues for deconv

c143b61

digantdesai force-pushed the deconv_pyre branch from 8d03078 to c143b61 Compare

January 17, 2025 20:29

Contributor

facebook-github-bot commented Jan 17, 2025

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


          Merge branch 'main' into deconv_pyre

ba8e2c7

Contributor

facebook-github-bot commented Jan 17, 2025

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

digantdesai closed this

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

Labels

CLA Signed module: xnnpack topic: not user facing