-
Notifications
You must be signed in to change notification settings - Fork 68
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Extension no longer shows up in Forge #256
Comments
+1 , excellent extension too. Hopefully we can figure it out. I get this at startup: Using sqlite file: /content/gdrive/MyDrive/sd/stable-diffusion-webui-forge/extensions/sd-webui-agent-scheduler/task_scheduler.sqlite3 |
probably because of Gradio 4.0 update |
I haven't updated forge recently and the extension has stopped working there too. It does still appear in the UI but pressing the "enqueue" button doesn't do anything - worked fine up until maybe a day or two ago. Weird how base queue systems aren't included in GUI's at this point considering its such a useful tool. |
Could be your history database got too big. Try deleting the task_scheduler.sqlite3
history file (it’s in the extension’s directory).
…On Thu, Aug 15, 2024 at 7:24 AM Miuna88 ***@***.***> wrote:
I haven't updated forge recently and the extension has stopped working
there too. It does still appear in the UI but pressing the "enqueue" button
doesn't do anything - worked fine up until maybe a day or two ago. Weird
how base queue systems aren't included in GUI's at this point considering
its such a useful tool.
—
Reply to this email directly, view it on GitHub
<#256 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A7BYZOYJZUSRJNNPLNY3PMLZRS23VAVCNFSM6AAAAABMQYDG6CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJRGM3TCNJXG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
One of the best extensions around. Hope it gets updated to support Gradio 4 on Forge. |
Forge needs a good built-in queue system. |
same here. I hope that we get update soon to use with forge |
I would like to get this working in Forge also |
ok - I got it working - but barely. Forked this repo and fixed a few issues: Caveats: Forge now does not automatically load a model - so this breaks the agent scheduler. Only workaround is to generate manually first and then the enqueue should work. I haven’t tested a lot but this patch is not perfect but hope to sort out the issues over the next few days and hopefully submit a pull request to this repo. Submitted a pull request but deleted as it was a half baked approach (disclaimer: first time figuring out Gradio etc) |
I too think this extension should be embedded by default into Forge. pleeeeease! |
This issue looks like a dupe of #253 |
I've tried this, all the buttons is there but nothing happend when I press enqueue. |
Hi - yes so you need to "generate" first - and only then the enqueue would work with future generations. This is because of the way Forge has been re-written, where it does not auto load models/checkpoints. I am hoping to find some time this week to fix the Enqueue button action so it starts working as soon as it is clicked. |
Hi, |
Yeah, @supersonic13 's fork doesn't work anymore, even after genning first. |
There was a Forge update recently that makes the fork no longer work. Since I only really experiment with Flux and don't really use any new features, I just revert to an old commit and use the non-forked version of the scheduler, and everything works just fine again. |
Which one did you use? |
I am a little bit perplexed, as I had been using a very old commit (lllyasviel/stable-diffusion-webui-forge@29be1da) and wanted to provide you with a newer one, as well as perhaps pinpoint the recent commit that broke the Enqueue button; unfortunately (fortunately?) I could not find it! Every commit I tested worked just fine, including the most recent commit, lllyasviel/stable-diffusion-webui-forge@f4d5e8c. With that commit and the forked version of the scheduler, I can schedule jobs just fine (I only tested queuing jobs after generating once). 🤷🏻♂️ EDIT: Okay this is strange. After doing some more generations on the latest commit, the Enqueue button stopped working at some point. Apparently it's not broken from the start, which is why my tests of doing one generation and then running a job kept working. I don't have any more time to figure out which commit causes this behavior, but if you want a working old version with no Flux support, switch to the first commit I mentioned, and be sure to also switch to the non-forked scheduler. |
Oh yeah, unfortunately I need one what works with Flux |
If you have time, you can try some older commits. I can tell you for a fact that commit lllyasviel/stable-diffusion-webui-forge@c2d290e is the first one where I noticed the Enqueue button was broken, so commits prior to that would be a good starting point. |
@hendra2222 @cupofwater1 @brknsoul hello - pushed some changes to the fork: I'm now able to enqueue straight away (without needing to generate first) and it all seems to work for me. |
@brknsoul hello. i'll try to do a clean install and test it out again. are there any console logs that show up when the Enqueue button is clicked ? |
Okay, it works, but only after I refresh the tab (Chrome, Windows 10). Might be a Chrome Thing ™ ;-) |
So I think on first launch the Generate button isn't set, therefore the Enqueue button cannot be bound: ERROR:root:Generate button not set; cannot bind enqueue button. But after the Generate button is used once it is set, then if the browser tab is refreshed the Enqueue button can bind...but I have no idea how to fix that :( |
@5n0wCr45h thanks for the logs. i can add some code to bind the button after a couple of more IDs are set. |
Any success? |
Hello. I tried to reproduce this on my PC but couldn't. I have added another check in the code, in any case. Can you update the forked extension and try it out please ? |
I should have time to try it tomorrow and will post the results. |
Initial testing looks promising. |
:( sorry, second test and same thing, must press the generation button first, reload the tab, then Enqueue works. |
That's not great. Would it be possible to provide any console logs you might have when that happens please ? |
Sorry, it will me a while before I can put time into this :( |
@supersonic13 I don't expect updates to this repository, so If you're interested in continuing development, could you consider opening issues in your fork? |
hi sure .. I've just updated the repo to include Issues .. Obviously, any PRs are also welcome .. |
Hi, it seems Agent Scheduler no longer works in Forge -- the extension tab and Enqueue button are gone.
Currently running this version: lllyasviel/stable-diffusion-webui-forge@aadc0f0
The text was updated successfully, but these errors were encountered: