Closed DarkVamprism closed 1 year ago
Huh I didn't think about this. I'll take a look when I have some time this weekend... Thanks for raising, sorry for the inconvenience.
No problem, your extension helps a lot when I can avoid this issue :) Thank you for your hard work!
On Wed, 29 Mar 2023 at 01:41, thomas @.***> wrote:
Huh I didn't think about this. I'll take a look when I have some time this weekend... Thanks for raising, sorry for the inconvenience.
— Reply to this email directly, view it on GitHub https://github.com/thomasasfk/sd-webui-aspect-ratio-helper/issues/36#issuecomment-1487346974, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEWFYRC2ZY2UUE7U66XZYGTW6MPEJANCNFSM6AAAAAAWK3OFY4 . You are receiving this because you authored the thread.Message ID: @.***>
In fact: it only needs to be a multiple of 8
Thanks all for raising this, I've pushed a quick fix to make the step back to 8 always. I think tomorrow I may make it configurable because there is still a valid use-case for non img2img not being a multiple of 8.
https://github.com/thomasasfk/sd-webui-aspect-ratio-helper/pull/37
Using this extension(which I love by the way) causes an error if I have a height and width not a multiple of 64, which usually happens if I lock the aspect to the image width+height.