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

[DIFFS EN+] fixes to phoneme replacements with LangCode #1275

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Cadlaxa
Copy link
Contributor

@Cadlaxa Cadlaxa commented Sep 9, 2024

No description provided.

@oxygen-dioxide
Copy link
Contributor

Not all voicebanks use multi-dict. Old voicebanks and English-only voicebanks don't use multi-dict.
I'll fix this for all g2p voicebanks in their base class.

@stakira
Copy link
Owner

stakira commented Dec 20, 2024

Not all voicebanks use multi-dict. Old voicebanks and English-only voicebanks don't use multi-dict. I'll fix this for all g2p voicebanks in their base class.

What's the current status of this issue now? Is this PR still needed? Or is it better now?

@Cadlaxa
Copy link
Contributor Author

Cadlaxa commented Dec 20, 2024

This pr fixes the issue with the language code update, cuz before it expects to use the multi-dict but now, it works without them (because of oxygen's update to phonetic hints without typing the language code)

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