-
Notifications
You must be signed in to change notification settings - Fork 4
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
Stable Diffusion Image Size #32
Comments
Could you not just ask SD to generate the image based on the known epd screen size ? |
Yep that's what it's supposed to do. It accepts only multiples of 64 so
normal process is request an image slightly too big and then crop it. My
assumption is it's resizing somewhere instead of cropping.
…On Sat, 21 Jan 2023, 11:13 am recrudesce, ***@***.***> wrote:
Could you not just ask SD to generate the image based on the known epd
screen size ?
—
Reply to this email directly, view it on GitHub
<#32 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AUTEK7DDDEVQZNBHCT63BQLWTMSYXANCNFSM6AAAAAAUCCFLLU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Issue is based on resize_image() in provider.py. This will resize the image to capture as much as possible of it on the screen, which works depending on the display, however if the factor of resize required for the short edge of the screen is smaller than the factor of resize required on the long edge, it will display black bars. Writing replacement function fit_image() to thumbnail on a factor, instead of just on longest edge. |
…ests for fit_image and resize_image each. #32
Should be fixed. Just testing on pi, then I'll bundle it into a release. |
Appears to work. Closing for now and releasing new version. |
Check Stable diffusion works without resize and crops instead. Try to recreate #26 (comment) .
The text was updated successfully, but these errors were encountered: