ljleb
ljleb
That should cover all the feedback. If you still have a bit of energy left for this, let me know if it works better now. I added a function `update_cn_script_in_processing`...
Yeah I realized it wasn't right. Please try latest commit. I think you can use this function now: ```python def update_cn_script_in_processing( p: processing.StableDiffusionProcessing, cn_units: List[ControlNetUnit], is_img2img: Optional[bool] = None, is_ui:...
~~Hmm maybe an off-by one. I'll test the whole thing before asking for you to check that it meets your requirements now.~~ Not an off by one, I can't reproduce....
Btw is `p.script_args` empty before the call to `update_cn_script_in_processing` on your side, or does it have already filled arguments? The code assumes all script args are already pre-filled with some...
Do you know if it's the scripts before or after that break? I assume it's the scripts that are listed after controlnet in `p.scripts.awlayson_scripts`.
Thanks a lot for your time by the way. It makes the api more suitable for everyone and I appreciate it!
run `.\venv\Scripts\activate` on windows. (at the root of the webui dir, not the extension)
This looks clean, I'll test to see if the conversation above should be marked as resolved.
This PR also normalizes the model names in config files to singular form, to minimize surprise when guessing config keys. Let me know if this is not wanted. Note that...
With recent changes it is possible that the code no longer works. I'll take a look when I find some time.