Skip to content
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

Revert "Add support for groupwise quantization for int8 weight only quantization" #1164

Closed
wants to merge 1 commit into from

Conversation

jerryzh168
Copy link
Contributor

Reverts #1121

turns out this is not needed

Copy link

pytorch-bot bot commented Oct 24, 2024

🔗 Helpful Links

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

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

✅ You can merge normally! (1 Unrelated Failure)

As of commit 04f6cd5 with merge base 58edb7e (image):

BROKEN TRUNK - The following job failed but were present on the merge base:

👉 Rebase onto the `viable/strict` branch to avoid these failures

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 Oct 24, 2024
@andrewor14 andrewor14 closed this Oct 24, 2024
@andrewor14
Copy link
Contributor

@jerryzh168 By the way we might still need this for int8 embedding. If that's not the case we can reopen this.

yanbing-j pushed a commit to yanbing-j/ao that referenced this pull request Dec 9, 2024
…ction (pytorch#1164)

* Remove unnecessary __dict__ access from ModelArgs.transformer_args

* Refix by casting to TransformerArgs
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants