search

AUTOMATIC1111 / stable-diffusion-webui

Stable Diffusion web UI
GNU Affero General Public License v3.0
144.22k stars 27.09k forks source link

Highres. fix issue #1592

Closed vgaggia closed 1 year ago

vgaggia commented 2 years ago

First of all, wow it makes a nice difference, but sadly it seems that it only works with euler a currently, is this a bug or will it not be implemented with the other samplers.

devilkkw commented 2 years ago

Tested, it work in all sampler at any size, except plms. i running in local. last build clean install.

vgaggia commented 2 years ago

You seem to be mostly correct, but on dpm fast and adaptive it does not work as well, turns out all the ones i use are the ones that don't work, other than euler A at least

devilkkw commented 2 years ago

really strange. maybe is problem about dimension? i tested on 512x512 and 1024x768, as i say it work well on all sampler except plms,where i get an error. you have last build? you tested on clean install? what system are you running?

vgaggia commented 2 years ago

I did a clean install the other day, i'm running a i9 10850k, 32gb of ram and a 3090, so it's not a hardware thing

vgaggia commented 2 years ago

The fact that you say it's working for you seriously confuses me because it really doesn't for me on those samplers i mentioned

devilkkw commented 2 years ago

i running in windows11 with miniconda in virtual enviremont. As i remember i had to manually install requirement, with "pip install -r requirements.txt" That only difference i have from official install guide.

mezotaken commented 1 year ago

https://github.com/AUTOMATIC1111/stable-diffusion-webui/commit/b186d44dcd0df9d127a663b297334a5bd8258b58