-
Notifications
You must be signed in to change notification settings - Fork 482
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
safetensors dependency #87
Comments
It's not supposed to be required, but it is currently because train_lora_dreambooth calls save_safeloras indiscriminately. I'll add a PR soon. My intent was that safetensors will be required to save safetensors, but not to load them (there's a pure python version), and that pytorch pickles would remain an option. (I'd personally prefer if safetensors became the default, but that's up to @cloneofsimo and the community generally). |
#101 should fix this |
This should be fixed now #101 is merged @oscarnevarezleal - feel free to reopen if something still breaks without safetensors installed. |
I believe #84 introduced a safetensors dependency but it's not specified in requirements.txt
The text was updated successfully, but these errors were encountered: