choey / Comfy-Topaz

Topaz Photo AI integration for ComfyUI
MIT License
26 stars 4 forks source link

Face recovery #7

Open light-and-ray opened 1 week ago

light-and-ray commented 1 week ago

Omg. I'm developing a similar extension for sd-webui, and stocked because I didn't understand how to disable face rocovery (it's awful in pair with upscaling. Then I found your extension, and understood you don't know how too. But while exploring, I got an idea use args'--face_recovery', 'enabled=false', and it works! It's sad there's no a proper docs about tpai.exe

ON 16989920111632 OFF 16989920111655

choey commented 1 week ago

Hi there! It's a pleasant surprise with a hint of amusement to find someone else going down the same path of reverse-engineering tpai.exe. Sounds like we share the same strategy of trial-and-error :) One thing I found somewhat helpful is using the GUI to explore capabilities/params/model names that the CLI might also expose. I haven't done much digging since producing the proof-of-concept, but if I were to, I'd try cracking tpai.exe open with a disassembler (e.g., maybe there are other "hidden" flags, etc.).

Thanks for sharing the info on face_recovery! I'll keep this open as a feature request to expose this option in the comfy node.