Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support of the XTTS 2 in tts-server #227

Closed
wants to merge 3 commits into from

Conversation

jbgi
Copy link

@jbgi jbgi commented Dec 20, 2024

Fix #99.

Based on original PR by @bivashy: coqui-ai#3476

@eginhard eginhard self-assigned this Dec 23, 2024
@eginhard
Copy link
Member

Cool, thanks! I'll test it.

@mikejgray
Copy link

FWIW, I built and tested this locally and have a Docker image at mjgray84/coqui:latest. Works great!

@eginhard
Copy link
Member

Hey @jbgi ! Thanks again for opening this. I looked into it for a bit and think that instead of this it would be best to adapt the server to use Coqui's Python API internally (which wasn't available at the time the server was originally created). This means the server can support all current and future models without further adaptation.

If you, or anyone else, would like to do this, it should be fairly easy to do. It would be very similar to how the command line interface is implemented in TTS/bin/synthesize.py, i.e. creating a TTS instead of a Synthesizer instance. Feel free to ask me any questions. Otherwise I can take care of it eventually.

@jbgi
Copy link
Author

jbgi commented Dec 29, 2024

If you, or anyone else, would like to do this, it should be fairly easy to do.

I'll leave it to someone else as I won't have the time to look into tts internals any time soon.

@eginhard
Copy link
Member

I looked into it for a bit and think that instead of this it would be best to adapt the server to use Coqui's Python API internally (which wasn't available at the time the server was originally created). This means the server can support all current and future models without further adaptation.

Just did that in #252, so closing this PR.

@eginhard eginhard closed this Jan 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] tts-server unable to start with xtts_v2 model
3 participants