-
Notifications
You must be signed in to change notification settings - Fork 992
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
Clip Skip nonfunctional with SDXL-based checkpoints #387
Comments
At least upstream, it was determined CLIP skip should not affect the output of any SDXL generations since all models were trained using the penultimate layer (which was not the case for SD1). AUTOMATIC1111/stable-diffusion-webui#12518 (comment) |
Oh. Well, that certainly explains it. Maybe it should be hidden from the UI when an SDXL model is loaded? It's at least a bit misleading the way it's currently displayed. |
i can fix it in 5 minutes but forge will try to get same result with webui however if most users vote for a functional sdxl clip skip but upstream refuse to have it, Forge may still implement it after multiple user reports because it can be seen as a part of backend thing |
Checklist
What happened?
All checkpoints based off Stable Diffusion XL, including the base checkpoint, do not show any variation when changing Clip Skip, even if you set it to 12.
Image generation parameters show that the changing Clip Skip value is being recognized, it shows up in the image info text after generation is complete, but the value doesn't actually affect the output at all.
Steps to reproduce the problem
Observe that Stable Diffusion checkpoint properly recognizes the Clip Skip parameter during image generation, but Stable Diffusion XL checkpoints do not.
What should have happened?
Clip Skip should modify the image output normally.
What browsers do you use to access the UI ?
Google Chrome
Sysinfo
sysinfo-2024-02-24-05-27.json
Console logs
Additional information
No response
The text was updated successfully, but these errors were encountered: