Skip to content

Moving Quant functions out to quant_lib.py: Part 1 #2846

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

Closed
wants to merge 2 commits into from

Conversation

Jack-Khuu
Copy link
Contributor

Summary:
Export_llama_lib is currently smothered in Quant related code. This diff starts a stack of refactors to move the code out of the export_llama_lib

Specifically this moves to quant_li.py (new), only the lines that do not require manual editing.

i.e. verbatim copy and paste


Note: This stack intentionally DOES NOT fix any existing style/refactor/feature. Those must come later otherwise, nothing gets landed

Differential Revision: D55723711

Copy link

pytorch-bot bot commented Apr 4, 2024

🔗 Helpful Links

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

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

✅ No Failures

As of commit 4f95b1b with merge base 7c7886b (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 Apr 4, 2024
@facebook-github-bot
Copy link
Contributor

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

Summary:

Removing GPTQ and Handlers in quantize.py in favor of AO

Reviewed By: jerryzh168

Differential Revision: D55554093
Summary:

Export_llama_lib is currently smothered in Quant related code. This diff starts a stack of refactors to move the code out of the export_llama_lib

Specifically this moves to quant_li.py (new), only the lines that do not require manual editing.

i.e. verbatim copy and paste

---
Note: This stack intentionally **DOES** **__NOT__** fix any existing style/refactor/feature. Those must come later otherwise, nothing gets landed

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

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

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in d9c09f4.

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 Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants