-
Notifications
You must be signed in to change notification settings - Fork 27.1k
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
[Feature Request]: SDXL refiner support #11919
Comments
maybe it can works like hi-res fix: |
Seems like a good idea but I would prefer if we could still access the original image to re run it with different seeds since the high res fix can and do cause some bad artifacts or good base images. |
I don't think what you said is a good idea. The best workflow should be base>refiner>hires. With the idea I proposed you can just select NONE on the refiner. Both problems solve with this. |
So you are against having a tick box in settings for cashing images that enable an non destructive workflow and I'm not sure if that workflow will be the best either since the upscaling can change things in weird ways or are you talking about the upscaler under extra? |
Idk what kind of upscaling you use. I never experienced what you mentioned. And no, I'm not against having a tickbox that can enable and disable the refiner. |
BTW. The cash is actually really helpful to find the issue also and is a debug tool in many situations. |
Personally I would prefer it as an img2img Script. I prefer sometimes to try and make several variations of the base image first before I go on to upscaling. I would like to have the same option available for SDXL refining as well. But I don't see why both options wouldn't be possible. |
@svenkevs you can already use it in img2img, but it's basically a misunderstanding the refiner model is not intended to be used to upscale/improve an image. |
that's one way to use the refiner, but the default design workflow is indeed img2img (SEdit) 1.0 has some new workflow, evidently. |
+1 for merging sdxl refiner with the base model - my own suggestion is to add it as a Script, which will allow unwilling people to ignore it and later img2img to get the refinement, and others will get it as part of the base, as SDXL predicts |
I think like the idea of the refiner and the selection of the refiner model being a part of HRF, or at least in the same UI space as HRF and as a new additional optional process. |
After gathering some more knowledge about SDXL and ComfyUI, and experimenting few days with both I've ended up with this basic (no upscaling) 2-stage (base + refiner) workflow: It works pretty well for me - I change dimensions, prompts, and samplers parameters, but the flow itself stays as it is. It might come handy as reference. UPDATE: after running some more experiments with various flows I've also managed to setup simple, yet nicely working (it doesn't sharpen areas that should stay blurred, for example) hires fix based on SDXL refiner - it looks like this: Would be nice to have something like that in SD web UI, too. |
This extension implements the refiner as intended by SAI. https://github.com/wcde/sd-webui-refiner |
Is there an existing issue for this?
What would your feature do ?
Implement the refiner not as img2img, like done on ComfyUI.
Proposed workflow
Add a dropdown menu for the refiner next to the checkpoint dropdown menu.
Additional information
No response
The text was updated successfully, but these errors were encountered: