ComfyUI icon indicating copy to clipboard operation
ComfyUI copied to clipboard

VAEDecodeTiled causes visible seams in tiling

Open cathodeDreams opened this issue 2 years ago • 3 comments

Hello. When generating large images I use the VAEDecodeTiled node to avoid CUDA memory errors, but in doing so it seems to result in an image that does contain visible seams where the tiling occurred. Are there steps I cant take to mitigate this or will it be inherent to using the node?

Here are two images showing the effect. They are simple 2-pass latent upscales.

ComfyUI_02583_

ComfyUI_02590_

cathodeDreams avatar Mar 19 '23 17:03 cathodeDreams

Yes, that's a known issue that I'm going to fix. This is the reason why it's in the _for_testing section.

comfyanonymous avatar Mar 19 '23 17:03 comfyanonymous

No rush, much to do. Didn't know if that was what specifically kept it in testing.

cathodeDreams avatar Mar 19 '23 17:03 cathodeDreams

Sounds like it's using Go Big type method, which if so, can't really be overcome without doing stuff like img2img color fix, which also imo, produces bad quality.

WASasquatch avatar Mar 19 '23 18:03 WASasquatch

If you update to the latest code it's still not perfect but it should be improved.

comfyanonymous avatar Mar 22 '23 08:03 comfyanonymous

It is significantly better comfy thank you. As of currently my issue is concluded, though I'm sure the node can continue to be improved.

ComfyUI_03850_

cathodeDreams avatar Mar 22 '23 16:03 cathodeDreams

That looks spectacular and rather continuous even when raising the levels of the image

WASasquatch avatar Mar 24 '23 08:03 WASasquatch