-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Correct hf hub download #1767
Correct hf hub download #1767
Conversation
The documentation is not available anymore as the PR was closed or merged. |
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.
Oh wow, for some reason I expected the HF hub to look at the variable too, thanks for the update!
* allow model download when no internet * up * make style
* allow model download when no internet * up * make style
The issue fixed in this PR has reappeared, and it was caused by this PR. It can be reproduced using the code here: #1717 (comment)
|
Hmm, I thought this was definitely fixed with #6456 and #6467 😕 Let me have a look at it. I'll try to reproduce with #1717 (comment) and report back. |
I have been to reproduce it with diffusers 0.25.0. In fact, this is indeed fixed by #6456 and #6467 but they haven't made it to the 0.25.0 release. I retried by installing So @patrickvonplaten would it be possible to either make a patch release @Sanster in the meantime, you can install |
Let's make a patch release for this I'd say (cc @sayakpaul wdyt?) |
Actually I think just doing |
Yes the patch release is needed to catch the |
Makes sense, sorry I misunderstood the previous comment a bit 😅 |
Make sure only local files are downloaded if
HF_HUB_OFFLINE
is set to a truthy value.