UI broken with Gradio 3.2x

akx commented 1 year ago

For end users: For the time being, I would recommend not updating AUTOMATIC1111/stable-diffusion-webui past e.g. a9eab236d7e8afa4d6205127904a385b2c43bb24 if you rely on this extension.

https://github.com/AUTOMATIC1111/stable-diffusion-webui/commit/9f0da9f6edfb9be1d69ba3492a61d96db769307b updates Gradio from 3.16.x to 3.22; this seems to break the JavaScript in this extension despite the fixes in #316:

Cannot read properties of null (reading 'append') TypeError: Cannot read properties of null (reading 'append')
    at dynamic_prompting.js?1679820957.2935863:15:49
    at runCallback (script.js?1679821875.4057686:38:9)
    at script.js?1679821875.4057686:44:31
    at Array.forEach (<anonymous>)
    at executeCallbacks (script.js?1679821875.4057686:44:11)
    at MutationObserver.<anonymous> (script.js?1679821875.4057686:56:9)

etc.

adieyal commented 1 year ago

@akx I can look into it later today if you aren't already working on it.

akx commented 1 year ago

@adieyal Please do! I just noticed this while working on the new wildcard stuff. I'm using an older webui for now :)

modulator commented 1 year ago

Yup - I think same issue for me! Basically Dynamic Prompts doesnt show up on Text2Img panel...

adieyal commented 1 year ago

I'm afraid that I can't get my webui to work with the newest version. I'm currently using 33b85391 which is more or less as far as I can go before it no longer runs on my setup. I have made some changes which fixed some versions here: https://github.com/adieyal/sd-dynamic-prompts/tree/bug/gradio3.2 but I can't tell whether that would fix the most recent webui.

rektobot commented 1 year ago

i noticed you closed the mac thread, you still need me to revert and run some tests? also of note, typically when auto1111 shits the bed after an update i cant even get dynamic-prompts to show up in the webui most the time without manual install cell before start-up, let alone run iterations. this past weekend after i installed ngrok for tunneling i was able to get usable batches out of it with exception of magic prompt being broken and throwing errors. i was doing combinatorials maxed out, 300 batches of 3 imgs and stuff and it didnt hiccup or throw any errors at all. in conclusion gradio is comped dogshit.

adieyal commented 1 year ago

Someone in that thread tested and said it worked. Let me know if it doesn't for you. Re-manual install, do you mean the dynamicprompts library or the sd-dynamic-prompts extension?

On Tue, 28 Mar 2023, 03:33 rektobot, @.***> wrote:

i noticed you closed the mac thread, you still need me to revert and run some tests? also of note, typically when auto1111 shits the bed after an update i cant even get dynamic-prompts to show up in the webui most the time without manual install cell before start-up, let alone run iterations. this past weekend after i installed ngrok for tunneling i was able to get usable batches out of it with exception of magic prompt being broken and throwing errors. i was doing combinatorials maxed out, 300 batches of 3 imgs and stuff and it didnt hiccup or throw any errors at all. in conclusion gradio is comped dogshit.

— Reply to this email directly, view it on GitHub https://github.com/adieyal/sd-dynamic-prompts/issues/338#issuecomment-1486042246, or unsubscribe https://github.com/notifications/unsubscribe-auth/AADGXMS3RUGFGNEZRP57ISTW6IWTXANCNFSM6AAAAAAWIDLLSY . You are receiving this because you were mentioned.Message ID: @.***>

adieyal commented 1 year ago

I have pushed a fix which works on my end. If anyone wants to test, please checkout to the bug/gradio3.2 branch

git checkout -b bug/gradio3.2
git pull https://github.com/adieyal/sd-dynamic-prompts.git bug/gradio3.2

You'll need to refresh your browser to test. Please let me know if it has fixed it on your setup.

rektobot commented 1 year ago

it seems to still be happening with the UI issue that a lot of people are experiencing with the gradio update. dynamic-prompts still runs every batch but now i cant open the menu, whenever i try the arrow tick to open & adjust settings it freezes the interface then it times out & i have to reload. when i reload i run a batch, my settings remain the same in dynamic prompts w/o magic prompt but i cant open the extension. it doesnt throw any errors after the initial batch error and puts the csv in the outputs directory with the imgs. i need to revert back to the last reasonably 'stable' commit anyway because a lot of stuff is throwing up errors now and the interface is all wonky, the preview screen is tiny, those stupid gradio animations freeze everything so i have to mess with the css file, hypernetworks decided to just do whatever the eff they want. this is the only error im getting out of dp now:

Traceback (most recent call last): File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 417, in process script.process(p, *script_args) File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/sd-dynamic-prompts/sd_dynamic_prompts/dynamic_prompting.py", line 388, in process GeneratorBuilder( TypeError: set_is_magic_prompt() got an unexpected keyword argument 'batch_size'

adieyal commented 1 year ago

Traceback (most recent call last): File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 417, in process script.process(p, *script_args) File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/sd-dynamic-prompts/sd_dynamic_prompts/dynamic_prompting.py", line 388, in process GeneratorBuilder( TypeError: set_is_magic_prompt() got an unexpected keyword argument 'batch_size'

I'm not sure what's going on with your slow down, but re the exception. Could you confirm that you're using the latest dynamicprompts library? If the interface lets you, go to the need help? section. The version is listed there. It should be v0.20.0. It should upgrade automatically but perhaps something prevented it from do it. If you're using an older version, the run python -m pip install dynamicprompts==0.20.0. I think that should sort at least one of your issues.

rektobot commented 1 year ago

heres what ive been doing recently and it seemed to work for the most past because updating from the webui was always a gamble at best and the cell from the readme in the repo worked if i edited it removing the magic prompts portion. now i just go to the main repo and look for the latest date and see if my files are up to date with it if not then i just c/p the raw files right into the right directory. i always manage to forget to do the requirements.txt tho. i noticed today my dynamic-prompting.py was still on 2.8.10 so thats probably the issue. i prefer updating from a cell on colab tbh, the webui is too finicky

Sent with Proton Mail secure email.

------- Original Message ------- On Tuesday, March 28th, 2023 at 11:54 AM, Adi Eyal @.***> wrote:

Traceback (most recent call last): File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/modules/scripts.py", line 417, in process script.process(p, *script_args) File "/content/gdrive/MyDrive/sd/stable-diffusion-webui/extensions/sd-dynamic-prompts/sd_dynamic_prompts/dynamic_prompting.py", line 388, in process GeneratorBuilder( TypeError: set_is_magic_prompt() got an unexpected keyword argument 'batch_size'

I'm not sure what's going on with your slow down, but re the exception. Could you confirm that you're using the latest dynamicprompts library? If the interface lets you, go to the need help? section. The version is listed there. It should be v0.20.0. It should upgrade automatically but perhaps something prevented it from do it. If you're using an older version, the run python -m pip install dynamicprompts==0.20.0. I think that should sort at least one of your issues.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

adieyal commented 1 year ago

heres what ive been doing recently and it seemed to work for the most past because updating from the webui was always a gamble at best and the cell from the readme in the repo worked if i edited it removing the magic prompts portion. now i just go to the main repo and look for the latest date and see if my files are up to date with it if not then i just c/p the raw files right into the right directory. i always manage to forget to do the requirements.txt tho. i noticed today my dynamic-prompting.py was still on 2.8.10 so thats probably the issue. i prefer updating from a cell on colab tbh, the webui is too finicky Sent with Proton Mail secure email.

In that case you should at least run something like:

!cd extensions/sd-dynamic-prompts
!git pull origin main

so that you don't have to copy files manually.

rektobot commented 1 year ago

perfect, thanks 👍🏻 i still dont understand the insistence on keeping gradio integration, it seems to be the cause of like 80% of the issues, at least on my end. there are other solutions. chatGPT shilled me fastAPI and another one i forget, he even went so far as to give me the code i would need to fully implement it in the current webui along with a separate conda environment and how to get it up and running to test it with auto1111. im like dude holy crap i just want faster tunneling solution cause gradio sucks lol. ngrok is a million times better so far btw

Sent with Proton Mail secure email.

------- Original Message ------- On Tuesday, March 28th, 2023 at 3:06 PM, Adi Eyal @.***> wrote:

heres what ive been doing recently and it seemed to work for the most past because updating from the webui was always a gamble at best and the cell from the readme in the repo worked if i edited it removing the magic prompts portion. now i just go to the main repo and look for the latest date and see if my files are up to date with it if not then i just c/p the raw files right into the right directory. i always manage to forget to do the requirements.txt tho. i noticed today my dynamic-prompting.py was still on 2.8.10 so thats probably the issue. i prefer updating from a cell on colab tbh, the webui is too finicky Sent with Proton Mail secure email.

In that case you should at least run something like:

!cd extensions/sd-dynamic-prompts !git pull origin main

so that you don't have to copy files manually.

— Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: @.***>

adieyal / sd-dynamic-prompts

UI broken with Gradio 3.2x #338