Add ability to use a remote SD backend

SillyTavern / SillyTavern-Extras

Extensions API for SillyTavern.

GNU Affero General Public License v3.0

549 stars 124 forks source link

Add ability to use a remote SD backend #7

Closed Babnam closed 1 year ago

Babnam commented 1 year ago

Changes

Ability to choose between running a local sd model or using a remote sd API backend
Made arguments for local and remote sd mutually exclusive
Added new requirement: sdwebuiapi
Updated the README with the new arguments and their descriptions

Related Issues

Closes #3

Cohee1207 commented 1 year ago

Splendid work, many thanks for the contribution!

I'm ready to merge it, just a couple of general comments: it would be great to set a sampler, a number of steps, and a model to use via the API call because Euler a is pretty much irrelevant at this point. Also would be neat to have an endpoint to get all the models from remote SD to let the user select it on the Tavern plugin UI (I hope this is coming soon).

Let me know if there are any questions.

Babnam commented 1 year ago

Thank you for your feedback. I have implemented the features you suggested, such as setting the sampler, steps, cfg scale and the model via the API call. I also added endpoints for getting available models and samplers, as well as the current model. The rest were minor changes such as handling any errors if a connection to the remote SD backend fails.

Please let me know if there's anything else :)

Cohee1207 commented 1 year ago

Merged, I will test the rest on my end