-
Notifications
You must be signed in to change notification settings - Fork 30.6k
[torchao safetensors] renaming get_state_dict function #40774
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
Conversation
ee90575
to
b171b2d
Compare
b15033e
to
9e5d89e
Compare
9e5d89e
to
e667169
Compare
cc @SunMarc |
cc @MekkCyber @SunMarc please take a look again |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks 🤗 LGTM
@MekkCyber do you mind merging this? thanks! |
[For maintainers] Suggested jobs to run (before merge) run-slow: mxfp4 |
Hi @liangel-02 ! will merge when the ci finishes 🤗 |
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice !
…0774) renaming get_state_dict function Co-authored-by: Mohamed Mekkouri <93391238+MekkCyber@users.noreply.github.com>
…0774) renaming get_state_dict function Co-authored-by: Mohamed Mekkouri <93391238+MekkCyber@users.noreply.github.com>
…0774) renaming get_state_dict function Co-authored-by: Mohamed Mekkouri <93391238+MekkCyber@users.noreply.github.com>
Summary
Renaming
get_state_dict()
method inhf_quantizer
toget_state_dict_and_metadata()
and updating functionality to return metadata along with state_dict.This reduces confusion related to enabling torchao safetensor support built out in this PR.
Test Plan
python -m pytest ./tests/quantization