stable-diffusion-webui icon indicating copy to clipboard operation
stable-diffusion-webui copied to clipboard

[BUG:] Reload script button broken

Open githubartman opened this issue 2 years ago • 2 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues and checked the recent builds/commits

What happened?

I cant exactly pinpoint at which commit this changed, but using the Script Reload Buttons does something with the script, but doesnt actually update it. It will show me the line I edited, but with the old error still present (although it is fixed?)

Also it would be nice to have some feedback in the console if Reloading, clicking feels weird if nothing happens visually

Thanks for the many updates Automatic, I love your repo

Steps to reproduce the problem

Bug out a script, make it generate an error, fix error, reload script body, error still there

What should have happened?

Reloading the script custom bodies

Commit where the problem happens

Latest

What platforms do you use to access UI ?

Windows

What browsers do you use to access the UI ?

Mozilla Firefox

Command Line Arguments

No response

Additional information, context and logs

No response

githubartman avatar Oct 30 '22 09:10 githubartman

I also encountered a similar problem. After using Restart Gradient and Refresh Components and restarting, the webui was restarted, but then the restart function failed, and there were many errors on the console. Snipaste_2022-10-30_23-06-05 The first time I found this error was after I pulled 35c45df28b303a05d56a13cb56d4046f08cf8c25 Now(17a2076f72562b428052ee3fc8c43d19c03ecd1e), this problem still exists And the #3247 fix some error in console,I look forward to merging it, although it has nothing to do with this mistake

sgmklp avatar Oct 30 '22 15:10 sgmklp

After some checking I found that this is a Gradio's bug. Is fixed in 3.8 (we have 3.5) but bumping the version seems to break at least the progress bar and the way we reload. I'm working on fixing but I believe I need @AUTOMATIC1111 blessing to do such a breaking change.

bamarillo avatar Oct 30 '22 16:10 bamarillo