ModelSurge / sd-webui-comfyui

An extension to integrate ComfyUI workflows into the Webui's pipeline
MIT License
516 stars 32 forks source link

fixes #147

Closed ljleb closed 1 year ago

ljleb commented 1 year ago
ljleb commented 1 year ago

PNG info is actually broken with some larger workflows. I'll look into it, but I'm not sure we can do something about it. For example, this workflow (it's in the metadata of the image):

workflow ![52944-425230810-1girl](https://github.com/ModelSurge/sd-webui-comfyui/assets/32277961/bd013cb0-4a9a-4b68-b00b-ad6dc519ffd2)
ljleb commented 1 year ago

Found a fix. The webui needs to fix their png info parsing with this regex instead:

\s*([\w ]+):\s*("(?:\\.|[^\\"])+"|[^,]*)(?:,|$)

https://github.com/AUTOMATIC1111/stable-diffusion-webui/blob/68f336bd994bed5442ad95bad6b6ad5564a5409a/modules/generation_parameters_copypaste.py#L12

We can patch this line from the extension for the time being.