-
Notifications
You must be signed in to change notification settings - Fork 370
F64 dtype #669
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
Comments
It seems like everyone is trying to reduce the model and keep the quality, but here it is)) |
have you looked into the related issue link, it would give more context. im not reducing the model size or manipulating it in any way. it errors simply if i use it with stable-diffusion.cpp showing errors of it not recognizing certain tensors |
@stduhpf, this can be fixed, maybe easily, since GGML / SD.CPP support F64 and I64 types: The function stable-diffusion.cpp/model.cpp Line 961 in 10c6501
should be appended with:
I tested on SD 1.1 (https://huggingface.co/CompVis/stable-diffusion-v1-1) model, its FP16 text encoder also has i64 tensor which breaks generation. Also, its unet tensors have "unet.*" naming, therefore line stable-diffusion.cpp/model.cpp Line 1618 in 10c6501
should be changed to:
Or else diffusion model weight type will be undetected (??). But I'm not sure if these 2 fixes are enough, because SD complained about tensor: |
Models with F64 type tensors do not work, specifically this SD 1.5 model: https://civitai.com/models/7371/rev-animated?modelVersionId=425083
-- even after adding a --type argument, it still fails
further research only led me to this other issue: #153
the offered program by the owner sadly did not work and i didnt want to fiddle with it any further
The text was updated successfully, but these errors were encountered: