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

Bug: self.model_name needed to be initialized. #2983

Merged
merged 2 commits into from
Oct 28, 2023
Merged

Conversation

vltmedia
Copy link
Contributor

Bug: self.model_name needed to be initialize to get around a bug that automatically crashes when the user provides the model paths but no model_name when initializing the TTS object. It literally just needs to be an empty string.

The error when no model_name was provided, only local cached models and config paths.

TTS/api.py", line 109, in is_multi_lingual
    if "xtts" in self.model_name

Bug: self.model_name needed to be initialize to get around a bug that automatically crashes when the user provides the model paths but no model_name when initializing the TTS object.
@CLAassistant
Copy link

CLAassistant commented Sep 23, 2023

CLA assistant check
All committers have signed the CLA.

@erogol
Copy link
Member

erogol commented Sep 25, 2023

@vltmedia you wanna sign the CLA?

@vltmedia
Copy link
Contributor Author

@erogol Yeah am down to sign it, where can I?

@erogol
Copy link
Member

erogol commented Oct 6, 2023

@vltmedia you can do it above where it says CLA not signed yet.

@vltmedia
Copy link
Contributor Author

@erogol sorry it took a while, just signed. The CLA badge/component wasn't rendering on my PCs, had to use my phone for it to render and be clickable.

@erogol erogol merged commit eccc94b into coqui-ai:dev Oct 28, 2023
48 checks passed
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.

3 participants