SillyTavern / SillyTavern-Extras

Extensions API for SillyTavern.
GNU Affero General Public License v3.0
549 stars 124 forks source link

DeprecationWarning: Nesting argument groups is deprecated #134

Open Sarcas666 opened 1 year ago

Sarcas666 commented 1 year ago

Because I wanted to move my installation to another drive, I decided to reinstall STExtras. I did it by the book; removed the old conda env, and recreated step by step following readme.md

Unfortunately, there is an error when I run STExtras:

(extras) C:\AIStuff\SillyTavern-extras>python server.py --listen --cuda --summarization-model=philschmid/bart-large-cnn-samsum --enable-modules=summarize,sd,chromadb
C:\AIStuff\SillyTavern-extras\server.py:91: DeprecationWarning: Nesting argument groups is deprecated.
  local_sd = sd_group.add_argument_group("sd-local")
C:\AIStuff\SillyTavern-extras\server.py:95: DeprecationWarning: Nesting argument groups is deprecated.
  remote_sd = sd_group.add_argument_group("sd-remote")

STExtras continues to run, but without sd functionality. There is no reaction, whether I use my local sd or the horde sd; nothing happens, not even an error.

I've reinstalled three times now, double-checking my steps. But the outcome is the same. I'm running stuff on W11/WSL/Python 3.11/and used the requirements-complete setup.

Thanks for any help.

Cohee1207 commented 1 year ago

Do you want to run SD remotely or in a built-in pipeline? Specify either --sd-local or --sd-remote flag Remote refers to AUTO1111 WebUI running in the API mode.

FryJay commented 1 year ago

Not sure if this applies to your situation but I saw this exact error when I accidentally updated my docker-compose file to use new features but forgot to rebuild the image before creating a new container.

Sarcas666 commented 1 year ago

@Cohee1207

Specify either --sd-local or --sd-remote flag I've reinstalled, but now with conda install python=3.10 and that fixed the problem. I'll try installing again one of these days with 3.11 and --sd-local. If that is required for 3.11, do update the install instructions in README.md

@FryJay Not sure if this applies to your situation I don't think so. I just followed the 'Locally'steps from the README.md and did not use docker. But thanks for the suggestion :)

Cohee1207 commented 1 year ago

I think running SD with Extras could be deprecated soon, as staging branch of ST now includes an ability to connect to AUTO1111 directly without Extras, and it provides much-much more control than using embedded Stable Diffusion pipeline in Extras