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

TTS: Add user dictionary parameter to tts-client #80

Merged

Conversation

manishaj-nv
Copy link
Contributor

This pull request introduces a addition of custom grapheme-to-phoneme dictionary for the Riva TTS client. The custom dictionary takes precedence over the default grapheme-to-phoneme rules and if a word is present in both the custom dictionary and the default rules, the custom mapping will be used.

Ensure custom handling of the dictionary is handled in the client end and is of the format key.
Read textFile and pass as string of the format to the API "ARON ˈdʒɑnsən,AARON ˈdʒɑnsən"

Usage
To use, pass the --user_dictionary flag with the path to your dictionary file when running the riva_tts_client

Example:
riva_tts_client --voice_name=English-US.Female-1 --text=" Hi Aaron, this is speech synthesizer" --audio_file=/opt/riva/riva-speech/output.wav --user_dictionary=<valid_dictionary_path>

@rmittal-github
Copy link
Contributor

@manishaj-nv let's also push a python-clients MR

@rmittal-github
Copy link
Contributor

@mohnishparmar as well for review

@rmittal-github
Copy link
Contributor

@manishaj-nv could you please resolve the open comments? also, need to update the latest proto SHA here and test the latest commit with CI

Copy link
Contributor

@rmittal-github rmittal-github left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, need to test with CI

@rmittal-github rmittal-github force-pushed the dev/mkj/tts_add_custom_g2p branch from 377678f to 2b28c05 Compare August 1, 2024 06:39
@rmittal-github rmittal-github force-pushed the dev/mkj/tts_add_custom_g2p branch from 171c072 to 97ec7ac Compare August 2, 2024 14:58
@rmittal-github rmittal-github merged commit 7364898 into nvidia-riva:main Aug 7, 2024
1 check 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