Skip to content

update language code for kokoro phonemizer - #212

Merged
katstankiewicz merged 1 commit into
mainfrom
pr/kokoro-lang-codes
Aug 18, 2026
Merged

update language code for kokoro phonemizer#212
katstankiewicz merged 1 commit into
mainfrom
pr/kokoro-lang-codes

Conversation

@katstankiewicz

Copy link
Copy Markdown
Collaborator

Main kokoro pipeline takes in "en-us" as a valid language code, but the phonemizer only accepts aliases like a for en-us

@raghavm243512

raghavm243512 commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator

the aliasing is done on the deployment side right? what phenomizer are you referring to

@katstankiewicz

katstankiewicz commented Aug 18, 2026

Copy link
Copy Markdown
Collaborator Author

the aliasing is done on the deployment side right? what phenomizer are you referring to

the aliasing is working properly for the main pipeline, but later on we see this error:

| File "/app/api/src/services/text_processing/phonemizer.py", line 82, in create_phonemizer
| raise ValueError(f"Unsupported language code: {language}")
| ValueError: Unsupported language code: en-us

I don't know where app/api/src/services/text_processing/phonemizer.py is, but as is, it only accepts the single letter language code

@raghavm243512 raghavm243512 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

not sure why this would have changed since we are using the same container, but this is for some odd reason what kokoro does

@katstankiewicz
katstankiewicz added this pull request to the merge queue Aug 18, 2026
Merged via the queue into main with commit e0041e3 Aug 18, 2026
2 checks passed
@katstankiewicz
katstankiewicz deleted the pr/kokoro-lang-codes branch August 18, 2026 17:13
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