-
Notifications
You must be signed in to change notification settings - Fork 26.9k
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
Hires fix change checkpoint #12181
Hires fix change checkpoint #12181
Conversation
This breaks in an ugly way if I use a lora. Could the loras perhaps be disabled, or maybe allow for a separate hires prompt? |
Ah yes, I had not noticed that. That will work. |
… is causing increased RAM usage
i get error for just setting below To create a public link, set
提示:Python 运行时抛出了一个异常。请检查疑难解答页面。 |
I'm not sure why this is happening @daxijiu, maybe an extension, but I pushed an edit that would prevent this from happening. |
How i pull this one? |
Any idea why this error pops up?
|
after update I got another error Python 3.10.8 (tags/v3.10.8:aaaf517, Oct 11 2022, 16:50:30) [MSC v.1933 64 bit (AMD64)] To create a public link, set |
All in same boat...
|
pushed an update for |
Hey, when i checkout the hires_checkpoint branch and launch with: |
Here is the part of the original post relevant to your question @spinferno:
|
There is an option in Settings -> User interface |
Great, it's working fine now! |
An error occurred after 4d9b096 Total progress: 100%|██████████████████████████████████████████████████████████████████| 30/30 [00:06<00:00, 4.84it/s] need --no-half ? |
rltgjqmcpgjadyd: can't run the webui right now but I pushed what I think should fix the problem. |
after c09bc2c It`s work! Great job! Thx! |
This is really good stuff, but something I think should be added is a two pass option, where you can decide the model and size for a hires fix, and then a refiner pass. Mainly because you would always want to use the refiner at 1x no matter what, but there could be a situation where you want to use normal hires (to upscale and add detail), but then use the refiner on a final pass after the initial hires fix. Honestly, I think "refining" should get it's own setting area since it is slightly different in purpose over hires fix and there are situations where you want to do hires fix and then refine. |
Can you please consider implementing not only the alternative two-stage SDXL img2img pipeline, but also the intended ensemble of experts pipeline for maximum quality? Here you don't run base to completion and renoise it before refining. Quote Stability AI:
https://huggingface.co/stabilityai/stable-diffusion-xl-base-1.0 |
i read it and it looks very promising i agree i hope we get this |
FWIW this extension already implements the aforementioned functionality. https://github.com/wcde/sd-webui-refiner |
While I do acknowledge the desire to switch models mid-sampling, and that's probably coming sometime soon, the official way to use the refiner from the official repo seems to be to just to txt2img and then run img2img on latents: |
The keyword is that it runs the refiner on the noisy latents.
It does not run a full txt2img pass thru all steps and then do an img2img pass like you describe. It purposefully stops sampling earlier on the txt2img pass if the refiner is enabled. I think it needs to be pretty explicitly stated that this PR is not the official way to use SDXL with the refiner. |
Description
Hires fix: show hires checkpoint and sampler selection
setting to see it.Checkpoints to cache in RAM
setting to speed it up.Screenshots/videos:
Checklist: