-
Notifications
You must be signed in to change notification settings - Fork 314
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
Support for shuttle-3-diffusion #469
Comments
In the version of ggml used by sdcpp, type 34 is invalid: https://github.com/ggerganov/ggml/blob/21d3a308fcb7f31cb9beceaeebad4fb622f3c337/include/ggml.h#L396 Shuttle-3-diffusion should work just like Flux Schnell in theory. Can you share the command you used to launch it? |
Thank you for your reply! |
FYI the real problem is in https://github.com/ggerganov/ggml/blob/21d3a308fcb7f31cb9beceaeebad4fb622f3c337/src/ggml.c#L1015, the entry for 34 is missing. |
I don't think shuttle-3-diffusion has built-in vae and text encoders. You should download t5xxl, clip_L (from here ) and the vae from the shuttle-3-diffusion repo, then use
I don't think it's related. Sdcpp uses an older version of GGML that doesn't support TQ1_0 type yet. This shouldn't cause any issues. |
Thank you for the pointers. I tried I believe sdcpp should at least imply that the process didn't finish. |
That's weird |
Okay it worked! |
I don't have a definitive answer, but it probably have something to do with the call to Glad you got it working though |
I downloaded the weights from https://huggingface.co/shuttleai/shuttle-3-diffusion, the program loaded the weights and exit for no error message.
I debugged the program, it seems that the problem is the call to
ggml_type_name()
with parameter 34, which is missing in the ggml.h file. Checking the llama.cpp repo, it should beGGML_TYPE_TQ1_0
, added from ggerganov/llama.cpp#8151The text was updated successfully, but these errors were encountered: