Skip to content

Don't partition per_tensor weights with qd8 (#8787) #8890

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

digantdesai
Copy link
Contributor

Summary:

This is not supported, so we shouldn't partition it.

Add an expectedFailure test to indicate that this is not supported.
ghstack-source-id: 269356867
exported-using-ghexport

Reviewed By: mcr229

Differential Revision: D70343584

@digantdesai digantdesai requested a review from mcr229 as a code owner March 3, 2025 19:37
Copy link

pytorch-bot bot commented Mar 3, 2025

🔗 Helpful Links

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

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

✅ No Failures

As of commit bc1a8b0 with merge base 9aca1fa (image):
💚 Looks good so far! There are no failures yet. 💚

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

@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 Mar 3, 2025
Summary:
Pull Request resolved: pytorch#8787

This is not supported, so we shouldn't partition it.

Add an expectedFailure test to indicate that this is not supported.
ghstack-source-id: 269356867
exported-using-ghexport

Reviewed By: mcr229

Differential Revision: D70343584
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D70343584

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D70343584

# for free to join this conversation on GitHub. Already have an account? # 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. fb-exported topic: not user facing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants