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

Add [DIFFS EN+] Phonemizer #1244

Merged
merged 3 commits into from
Sep 3, 2024
Merged

Add [DIFFS EN+] Phonemizer #1244

merged 3 commits into from
Sep 3, 2024

Conversation

Cadlaxa
Copy link
Contributor

@Cadlaxa Cadlaxa commented Aug 17, 2024

This phonemizer for Diffsinger uses ARPA+ G2p so the g2p features are the same with the ARPAsing+ Phonemizer

  • G2p Features
    • Support for dr, tr, dx ,ax and q (but q is replaced with cl and vf (phonemizer feature))
    • ' prefix on notes triggers glottal stop, if added to a vowel and the prevNeighbour is null, vf is used instead of cl
    • ' suffix for Manual relaxed consonants, (applicable only if the word ends with t or d)
  • Phonemizer features
    • Conditional replacements for q phoneme because the g2p uses q instead of cl
    • Automatic replacements of t/d to dx
    • Automatic replacements for n dx sequence and replace it with n, the actual phoneme for this is nx like (winner [w ih nx er])

@Cadlaxa
Copy link
Contributor Author

Cadlaxa commented Aug 27, 2024

the recent pr is dependent on #1248 so it errored out

@stakira
Copy link
Owner

stakira commented Sep 1, 2024

tests failing?

@stakira stakira merged commit 84ae7e1 into stakira:master Sep 3, 2024
3 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.

2 participants