Skip to content

delete v1 of Int8WeightOnlyConfig#4020

Closed
vkuzo wants to merge 3 commits intogh/vkuzo/230/headfrom
gh/vkuzo/231/head
Closed

delete v1 of Int8WeightOnlyConfig#4020
vkuzo wants to merge 3 commits intogh/vkuzo/230/headfrom
gh/vkuzo/231/head

Conversation

@vkuzo
Copy link
Copy Markdown
Contributor

@vkuzo vkuzo commented Mar 6, 2026

Summary:

Context: #2752

99% Claude

Test Plan: CI

@vkuzo
Update
9f6624e 
[ghstack-poisoned]
@vkuzo
Copy link
Copy Markdown
Contributor Author

vkuzo commented Mar 6, 2026 •
edited
Loading

Stack from ghstack (oldest at bottom):

@pytorch-bot
Copy link
Copy Markdown

pytorch-bot Bot commented Mar 6, 2026 •
edited
Loading

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/ao/4020

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

❌ 10 New Failures

As of commit 4c29e29 with merge base 637c4ac (image):

NEW FAILURES - The following jobs have failed:

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 Mar 6, 2026
vkuzo added a commit that referenced this pull request Mar 6, 2026
@vkuzo
delete v1 of Int8WeightOnlyConfig
2d5c2cb 
Summary:

Context: #2752

99% Claude

Test Plan: CI
ghstack-source-id: dbb6cfe
ghstack-comment-id: 4013955531
Pull-Request: #4020
This was referenced Mar 6, 2026
Merged
Closed
@vkuzo vkuzo added the module: inference quantize_ api inference flow label Mar 6, 2026
@vkuzo vkuzo requested review from andrewor14 and jcaip March 6, 2026 20:27
andrewor14
andrewor14 approved these changes Mar 9, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@andrewor14 andrewor14 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

also cc @metascroy, do you know of any internal uses of this config (version 1)?

@vkuzo
Update
2daca51 
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Mar 17, 2026
@vkuzo
delete v1 of Int8WeightOnlyConfig
071ff4d 
Summary:

Context: #2752

99% Claude

Test Plan: CI
ghstack-source-id: f16b9ee
ghstack-comment-id: 4013955531
Pull-Request: #4020
jerryzh168
jerryzh168 approved these changes Mar 18, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@jerryzh168 jerryzh168 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, please make sure CI is green before merging

@vkuzo
Update
4c29e29 
[ghstack-poisoned]
vkuzo added a commit that referenced this pull request Mar 19, 2026
@vkuzo
delete v1 of Int8WeightOnlyConfig
7e92645 
Summary:

Context: #2752

99% Claude

Test Plan: CI
ghstack-source-id: f16b9ee
ghstack-comment-id: 4013955531
Pull-Request: #4020
@jerryzh168 jerryzh168 mentioned this pull request Mar 19, 2026
Closed
17 tasks
@vkuzo
Copy link
Copy Markdown
Contributor Author

vkuzo commented Apr 14, 2026

this was done in #4151

@vkuzo vkuzo closed this Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrewor14 andrewor14 andrewor14 approved these changes

@jerryzh168 jerryzh168 jerryzh168 approved these changes

@jcaip jcaip Awaiting requested review from jcaip

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. module: inference quantize_ api inference flow

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@vkuzo @andrewor14 @jerryzh168