-
Notifications
You must be signed in to change notification settings - Fork 30.8k
[from_pretrained
] Small refactor from_pretrained
: move around unrelated stuff
#41445
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
from_pretrained
] Refactor from_pretrained
: unbloat, separate and simplify?from_pretrained
] Small efactor from_pretrained
: move around unrelated stuff
from_pretrained
] Small efactor from_pretrained
: move around unrelated stufffrom_pretrained
] Small refactor from_pretrained
: move around unrelated stuff
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, will allow us to have much better visibility over what's happening!
Co-authored-by: Cyril Vallez <cyril.vallez@huggingface.co>
…ormers into dynamic-weight-loader
…ormers into dynamic-weight-loader
cd32b80
to
155b44e
Compare
e8e48ef
to
06515b6
Compare
[For maintainers] Suggested jobs to run (before merge) run-slow: fbgemm_fp8, finegrained_fp8 |
Can you really delete |
Oh, fixed by #41613, I see. |
What does this PR do?
As we are moving towards having a dynamic weight loader, we need to have a simple
from_pretrained
call.This should help