Make a basic implement for webui plugin

[jtydhr88]

hi Stability team,
I am not sure you guys already start to implement the webui plugin or not, but I make a basic version for this big demand.
please let me know how you think about it.

[cruhl]

Hello! 👋

We haven't been able to pick this up yet, so I'd be extremely happy to help you get it finished and merged!

[jtydhr88]

Thanks! I make a TODO list in the issue #25 and readme, please feel free to request any features I missed.

[cruhl]

How would you like us to cooperate with you?

Should we create a branch which pulls in from yours, commit directly to yours, etc.?

[jtydhr88]

I think it is ok to commit directly, shall I add someone into my repo?

[KAJdev]

@cruhl should be as simple as gh pr checkout 24

[KAJdev]

Been playing around with this plugin, and so far it seems to work quite well, although with some major issues that need to be addressed before it can be merged:

issues

• img2img / masking is not implemented
• model/sampler seems to lose its state often
• seed seems to be ignored (not sure if this is webui automatically incrementing seed or not)

possible implementations:

• history (might be able to implement via https://developer.mozilla.org/en-US/docs/Web/API/File_System_Access_API)
  • can pull from a path in settings
• currently webui needs to be started manually with extra CLI flags.
  • plugin could use path in settings along with a field for extra cli flags to launch webui on startup
  • webui status could replace images generated status

[budaloops]

@KAJdev hey at least its a start right?

[jtydhr88]

Thanks for @KAJdev testing, let me add the requirements in the TODO list

[cruhl]

@KAJdev @jtydhr88 @daryllaro Taking a crack at this now

[merodiro]

possible implementations:

• history (might be able to implement via https://developer.mozilla.org/en-US/docs/Web/API/File_System_Access_API)

  • can pull from a path in settings

• currently webui needs to be started manually with extra CLI flags.

  • plugin could use path in settings along with a field for extra cli flags to launch webui on startup

Wouldn't these two implementations work only in case both StableStudio and SD-webui are on the same host and wouldn't support remote webui?

[jtydhr88]

@merodiro since this plug-in is just at start phase, I only have local environment for dev, once we finish the main features, we can test remote host. However in theory, it should support remote host easily

[merodiro]

I just tried it with remote sd-webui and it worked well. The only problem I had was that I couldn't load the samplers I think I was doing something wrong. it's loading them fine now

[merodiro]

When I don't choose a model it displays SDXL as the model used for the generation

[KAJdev]

I believe this is because internally StableStudio defaults to stable-diffusion-xl-beta-v2-2-2 as a model ID for input objects. If you aren't returning one it will just use the default object specified in Generation.Image.Model.StableDiffusionV1.Input.initial. This can be overridden by the plugin providing a default input.

[cruhl]

@jtydhr88 Are you cool with the README changes I committed? @KAJdev is making some tweaks to the plugin itself and then we're cool to merge. 🦾

[jtydhr88]

Yeah! Readme is good for me!

[KAJdev]

@jtydhr88 Refactor is going well, noticing some issues with the history plugin. Looks like get_root_path() is failing since __file__ doesn't exist in exec()ed python code. os.getcwd() should work though.

[jtydhr88]

@KAJdev thanks for catching, I will fix it soon

[jtydhr88]

hi @KAJdev as my test, there is an issue happening on get history image, even webui returns the data, but StableSudio sitll keeps loading without show image:

I output the data, and they are all Prmoise:

I have to revert your code regarding this part to my previous one.

Another issue is the status keeps on "Ready without history plugin" even the extension already be here and orks well to return data. (I don't fix it)

[KAJdev]

Yes, I was having issues with the history plugin, the error mentioned before as well as some possible CORS issues. Will investigate more.

Good catch on the async map there, that revert is fine.

[jtydhr88]

besides this issue, anything else needs to do for this init version? or i am good to merge

[KAJdev]

I think we are aiming for an early morning merge.

[cruhl]

@jtydhr88 Are we good to merge this from your perspective?

[jtydhr88]

@cruhl yes!

[SeriousM]

I really need to thank you @jtydhr88 (@cruhl , @KAJdev ) for bringing this to StableStudio. StableDiffusion now feels so much polished and slick!
Of course there are plenty of opportunities for more features like cancel current dreaming, progress bar/eta, set image size, set settings for variants before dreaming... but this first, very important step was really great work. Well done!