-
Notifications
You must be signed in to change notification settings - Fork 163
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
images came out segmented with squares/rectengles #111
Comments
me too, how to reverted back to version 1.3.2, help me please. |
Using I don't know internals much, so I'm unsure how it broke upscale |
Hi there, I'm the author of AUTOMATIC1111/stable-diffusion-webui#10295, which it sounds like might have inadvertently caused this. I've never used the Ultimate Upscale extension (it looks really cool though -- I'll probably use it now that I know it exists), so I'm not quite sure how to reproduce this on my end. Any chance you could provide an input image and settings for me to reproduce this on my end? Alternatively, if you're willing to try fiddling with the code based on instructions I can provide, maybe your test results can help me figure out what might be wrong. |
I can't use SD upscale anymore Output: Basic settings (just changed scale to 4 instead of 2) |
@cian0 Can you provide the Git commit hashes of the WebUI that you tried, which worked and didn't work? |
@Splendide-Imaginarius 741bd71873ebb16f61d36e4c8c0e64214501a638 works Img2img No prompts. Model: revAnimated_v122 There seems to be issues at edges of tiles where they merge. |
Thanks @eunnone, I'll try with those settings and see if I can reproduce it here. It may take me a few days. |
I can reproduce the issue now; it does indeed appear that 1503af60b0550d1a57c95a500df1fe1390178a27 broke it. I'm not yet sure why that commit broke it; I will investigate further over the next few days. Thanks again to @eunnone for helping me reproduce it. |
No, it's not this commit. Somewhere else is a problem. Tried reverting it but the problem didn't go away |
I have a candidate fix. It definitely fixes the broken behavior that @eunnone described, but I need a few more days to make sure it doesn't break outpainting (which was what 1503af60b0550d1a57c95a500df1fe1390178a27 was originally intended to fix).
@Jopezzia it wouldn't surprise me if there are multiple bugs somewhere, and 1503af60b0550d1a57c95a500df1fe1390178a27 only caused one of them. After some investigation, I definitely see what's going wrong in 1503af60b0550d1a57c95a500df1fe1390178a27, it's a legit bug. (Also it's subtle enough that I definitely see why it evaded my notice when I wrote that commit, and why it didn't get caught in upstream PR review.) |
Fixes breakage when mask_blur is set after construction. See Coyote-A/ultimate-upscale-for-automatic1111#111 (comment) Thanks to Алексей Трофимов and eunnone for reporting the issue.
@eunnone Could you please try this branch and let me know how it goes? https://github.com/Splendide-Imaginarius/stable-diffusion-webui/tree/mask-blur-property |
@Splendide-Imaginarius tried your branch now. Everything works) |
@Splendide-Imaginarius 2023-08-03_12-25-08.mp41.3.2 upscale is closer to the original image compared to the fixed version. |
Good to hear that we're moving in the right direction. Thanks @eunnone for pointing out the remaining breakage; it tentatively looks like I can reproduce that issue here. I'll see if I can diagnose the cause; may take a few more days. |
I have a lead on what's causing the remaining breakage; hoping to have a candidate fix ready in a few days. |
This more closely matches the old behavior of PIL's Gaussian blur, and fixes breakage when tiling. See Coyote-A/ultimate-upscale-for-automatic1111#111 (comment) Thanks to Алексей Трофимов and eunnone for reporting the issue.
@eunnone Could you give this branch a try? https://github.com/Splendide-Imaginarius/stable-diffusion-webui/tree/mask-blur-property+kernel |
@Splendide-Imaginarius Tested with multiple images. Works flawlessly now. The upscales are identical to the ones in 1.3.2. Thank you for the fix ! :) |
@eunnone Great, thanks for testing! I'll submit the fix upstream in the next few days. |
@eunnone Could you confirm whether the upstream |
@Splendide-Imaginarius Upscales look a bit worse on dev branch. There seems to be some smoothing/blurring. I got this controlnet error(?), might not be related
|
@eunnone Hmm, interesting. My |
It seems this commit causing the blur issue. Test for blur https://imgur.com/a/P9TfAnK (images and prompts) |
Are you using PLMS, DDIM, or UniPC? If not that's especially concerning. If you can post the original images used and created with PNG info (from both versions if you can) as well as your sysinfo from the settings page I can try looking a bit more into it. Edit: I see you posted your prompts in #111 (comment). Assuming that's up-to-date I'll see if I can work off that. Since it isn't using one of the aforementioned samplers something certainly seems to have gone wrong. Edit 2: Can confirm using the commit before the CFG denoiser impl (AUTOMATIC1111/stable-diffusion-webui@2a72d76) + merged with https://github.com/Splendide-Imaginarius/stable-diffusion-webui/tree/mask-blur-property+kernel, it produces the correct, non-blurry result. Using the same setup but with the commit that breaks it (AUTOMATIC1111/stable-diffusion-webui@8285a14) produces the blurry result. I'll see what's causing this exactly now. Edit 3: Figured it out. There should be a PR shortly. |
@eunnone Should be fixed as of AUTOMATIC1111/stable-diffusion-webui@c1a31ec. I can confirm that fixes it on my end. The results are now sharp as they were before. |
@catboxanon Thank you for the fix. Can confirm upscales are working perfectly now. |
some of the images came out like its been segmented. its like the whole picture is divided into equal squares/tiles. what did I do wrong? Have I been setting the denoising strength for too much? ( i set mine to 0.4 denoise)
The text was updated successfully, but these errors were encountered: