Update server.py to respect sd-remote

SillyTavern / SillyTavern-Extras

Extensions API for SillyTavern.

GNU Affero General Public License v3.0

563 stars 133 forks source link

Closed Dalethium closed 10 months ago

Dalethium commented 1 year ago

Currently if you don't specify a model for sd, it assumes you want remote SD instead of loading the default SD. This fixes that.

Cohee1207 commented 1 year ago

I think it will be a habit breaker for existing scripts of people using the SD module.

Dalethium commented 1 year ago

In that case maybe we could change how this is setup somehow? Currently executing the command does not behave as documented.

Cohee1207 commented 1 year ago

Maybe it's better to update the feature flag documentation instead, to reflect the actual behavior of the SD module?

Dalethium commented 1 year ago

That sounds fine! Whatever you feel is best. Feel free to close this PR if you end up doing that, I'll leave it open for now!