search

ljleb / sd-webui-freeu

a1111 implementation of https://github.com/ChenyangSi/FreeU
MIT License
301 stars 16 forks source link

Update default parameters to SD1.4 recommendations #3

Closed lewdogk closed 11 months ago

ljleb commented 11 months ago

You think the bounds are good (0 to 2) or we should change them as well?

lewdogk commented 11 months ago

It's probably best to give them a wide range to experiment with since this is very new stuff.

ljleb commented 11 months ago

Okay I'll merge and update the bounds to go from -1 to 3. we can always change the titles of the sliders later to be able to change the bounds.

Miczu commented 11 months ago

Now I'm interested to see what happens with negative values...

Nyxeka commented 11 months ago

make sure to check for --no-half before doing .to(32) and .to(16)

ljleb commented 11 months ago

The fft needs to work in fp32. I just pushed a fix that reverts to the original precision after the fft

Miczu commented 11 months ago

image Looks like once I loaded original version, the old values persists. Probably can fix it by deleting it from config... Update/reinstall didn't do it.

Miczu commented 11 months ago

"customscript/main.py/txt2img/Enable/visible": true, "customscript/main.py/txt2img/Enable/value": false, Probably needs better name for checkbox.

Miczu commented 11 months ago

Deleting those line from ui-config fixed it.

ljleb commented 11 months ago

Yeah you can delete the lines. We can also rename the sliders later if we end up shrinking the sliders range.