Skip to content

Conversation

rohansjoshi
Copy link
Contributor

Summary: Added bias rotation. This is needed to apply SpinQuant R2 to models which have bias such as Qwen models.

Differential Revision: D81352249

Summary: Added bias rotation. This is needed to apply SpinQuant R2 to models which have bias such as Qwen models.

Differential Revision: D81352249
Copy link

pytorch-bot bot commented Aug 30, 2025

🔗 Helpful Links

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

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

❗ 1 Active SEVs

There are 1 currently active SEVs. If your PR is affected, please view them below:

✅ No Failures

As of commit f0ac61c with merge base 568c193 (image):
💚 Looks good so far! There are no failures yet. 💚

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 Aug 30, 2025
@facebook-github-bot
Copy link
Contributor

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

@rohansjoshi rohansjoshi added the topic: improvement Use this tag if this PR is an improvement (doesn't fit into any of the other categories) label Aug 30, 2025
Copy link
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.

Thanks! Can you also add a quick unit test for these models? (can be separate PR)

@rohansjoshi rohansjoshi merged commit 71bfccb into pytorch:main Sep 2, 2025
18 of 22 checks passed
@rohansjoshi rohansjoshi deleted the export-D81352249 branch September 2, 2025 16:12
@jerryzh168
Copy link
Contributor

jerryzh168 commented Sep 2, 2025

I have the impression that spinquant implementation in torchao doesn't work before because it didn't improve accuracy, wondering if this would help fix the issue

previous accuracy test results can be found in #983 I think

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. fb-exported topic: improvement Use this tag if this PR is an improvement (doesn't fit into any of the other categories)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants