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

Update the list of supported phoneme ASR alignment languages. #522

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xennygrimmato
Copy link

The list of supported phoneme ASR alignment models can be found here: https://github.com/m-bain/whisperX/blob/main/whisperx/alignment.py#L24-L54

@deep-pipeline
Copy link

@m-bain This is an uncontroversial README improvement suggestion but the language list update is now a little out of date (from Oct 2023) if one looks at the relevant section of code linked to above..

PR #757 is probably a better approach and if incorporated would render this PR worth closing un-actioned. It might be encouraging of contributions to merge this and then immediately merge #757 (or else just reject this here and leave a thank you note for the original contributor?).

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.

None yet

2 participants