flowtyone / ComfyUI-Flowty-LDSR

LDSR custom node for ComfyUI
GNU General Public License v3.0
190 stars 14 forks source link

Only compatible with 1:1 ratio #2

Closed Botoni closed 8 months ago

Botoni commented 8 months ago

It seems that other resolution ratios than 1:1 results in this black spaces around the image.

1182:886 is the original resolution, a 4:3 ratio for sdxl recommended resolutions. The same happens with 16:9 ratios for example.

image

ltdrdata commented 8 months ago

I also observed the same symptom. However, it seems to be not a 1:1 ratio but likely a requirement for multiples of 64.

ltdrdata commented 8 months ago

I just made PR.

https://github.com/flowtyone/ComfyUI-Flowty-LDSR/pull/3

flowtyone commented 8 months ago

@ltdrdata looks great, thank you! Do you mind adding to that pr the updated workflow image?

stephantual commented 8 months ago

Weird I hadn't noticed that - good catch! @Botoni !

ltdrdata commented 8 months ago

@ltdrdata looks great, thank you! Do you mind adding to that pr the updated workflow image?

That's OK.

flowtyone commented 8 months ago

Merged fix from @ltdrdata , closing this issue