search

oobabooga / text-generation-webui-extensions

561 stars 108 forks source link

add XTTSv2 #50

Closed kanttouchthis closed 11 months ago

oobabooga commented 11 months ago

Could you submit the extension code to the main repository as a PR? I want to add this as a built-in.

kanttouchthis commented 11 months ago

Could you submit the extension code to the main repository as a PR? I want to add this as a built-in.

I'm working on that. Only problem is TTS requires --no-dependencies in order to work with the webui requirements and afaik you can't add --no-dependencies in a requirements.txt. would a bat / sh script be ok for installation?

oobabooga commented 11 months ago

The extensions requirements are installed before the web ui requirements in new installs. I assume some of the coqui requirements are overwritten by this, but maybe it will work anyway?

For existing installs, I could add a warning in the extension code in a try: import ... except: block with the correct installation command for people to copy and paste.

kanttouchthis commented 11 months ago

the issue is that coqui will install specific versions of many libraries, which would then be overwritten by the web ui requirements, causing longer install time and potential version conflicts. I have only been able to get it to work by manually installing TTS' requirements without version constraints and then installing TTS

kanttouchthis commented 11 months ago

I tested it and it works on windows at least. I made a PR

oobabooga commented 11 months ago

Thanl you! I'll try to test it tomorrow.