Closed eadnams22 closed 1 week ago
I would think a solution would be to allow us to select a (default) sampler, either as default, or via the txt2img api.
or just “same as primary” being the default.
it doesn’t fix the UniPC problem specifically for me, but it does allow me to bypass this odd bug.
issue is that for txt2img
api requests, denoising_strength is not set so its interpreted as 0 and that causes face-hires to fail.
add denoising_strength
value to your api request or set it globally in settings -> postprocessing -> face restore strength.
regarding sampler, set hr_sampler_name
in api request as well.
core issue is that api needs exact params. you're sending txt2img api request, but face-hires is internally img2img operation, so it gets triggered with default values not set by api server since it only initialized defaults for txt2img from original api request.
i'll add better handling of defaults for api in the future, but that will be handled as a feature item.
Issue Description
It all executes, then fails on the last step it seems.
It works fine if I use "restore" in the GUI, and generate that.
API content:
Upon further testing its because its defaulting to UniPC when the FaceHires (inpainting) sampler type comes up as "none". Does the same when inpainting with unipc in the GUI.
Version Platform Description
Fresh install. Tried Dev (and Master) on Linux (WSL2) using python 3.10
Did a fresh git pull, and rebuilt the venv for both as well.
Relevant log output
Backend
Diffusers
UI
API
Branch
Dev
Model
Other
Acknowledgements