-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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 #3476
Conversation
@bivashy I tried your changes on my end with a fine tuned model (xtts v2) and I could not get it to work. Were you successful using a fine tuned model? I'm running it like: And receive the following error: edit: full error: |
Hello, i am sorry to hear that. I have tested it only with model_path, but without config_path UPD: I will try to solve the problem saturday and sunday |
Try to run like this: |
Hey @bivashy , just tried that way and I receive the following error: |
Maybe we should move into the discussions? |
Putting this here in case someone else gets stuck, I was able to get things working for my setup: |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. You might also look our discussion channels. |
@bivashy I've tried what you've changed but still get errors server.py
xtts.py
manage.py
After running
|
Related issues: #3454 #3271 #3140
Description
This Pull Request changes behaviour of
Xtts#load_checkpoint
, andModelManager#download_model
.Changes:
ModelManager
for properconfig_path
resolution intts-server
checkpoint_path
,checkpoint_dir
parameters to follow theBaseTrainerModel
implementationFixes:
Current state
This Pull Request is tested with xtts2 model, and without specifying xtts2 model, it works fine.
Have a nice day!