search

Kilvoctu / aiyabot

A neat Discord bot for AUTOMATIC1111's Web UI
GNU General Public License v2.0
305 stars 79 forks source link

Update allowed height/widths #236

Closed JustGuyThatGuy closed 4 months ago

JustGuyThatGuy commented 5 months ago

Since December 2022, automatic1111 has supported resolutions that are a multiple of 8, not 64 This PR updates aiyabot to allow the same

sebaxakerhtc commented 5 months ago

@JustGuyThatGuy Bad idea... We moved from 32 to 64 because of resolutions list. In my fork I use minimum 768 res to have a list up to 2304px (Do you know the list of 25 is a maximum for Discord?) So... If we use 8 and minimum res by default 192 - then max will be 384px. Is it a good resolution for you?

JustGuyThatGuy commented 5 months ago

@sebaxakerhtc That is indeed what it'll show to start with, but when you start typing a number the list updates, e.g.

Sizes

Works perfectly on Desktop and Mobile Discord, so there's no reason to be so restrictive.

And realistically, who's actually doing 192px gens anyway unless they're trying to run SD on a potato? A more sensible default start might be 512.

Kilvoctu commented 4 months ago

Haven't tested this, but sounds reasonable to me. Merged