Skip to content

Add WER metrics: Faster-Whisper, NeMo ASR, Facebook’s HuBERT-large-finetuned#44

Open
whr-a wants to merge 6 commits intowavlab-speech:mainfrom
whr-a:main
Open

Add WER metrics: Faster-Whisper, NeMo ASR, Facebook’s HuBERT-large-finetuned#44
whr-a wants to merge 6 commits intowavlab-speech:mainfrom
whr-a:main

Conversation

@whr-a
Copy link

@whr-a whr-a commented Jun 23, 2025

This update adds three new WER metrics computed by different ASR models:

setup.py Outdated
"espnet_model_zoo",
"discrete-speech-metrics @ git+https://github.com/ftshijt/DiscreteSpeechMetrics.git@v1.0.2",
"cdpam",
"nemo_toolkit[asr]"
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for fixing this. I'm a bit conservative to put nemo here as they are not always keep the latest related to the other packages version and it is difficult to keep them aligned all the time. Would be better to put it in tools as additional installers

setup.py Outdated
"importlib-metadata",
"kaggle",
"kaldiio",
"jamo",
Copy link
Contributor

Choose a reason for hiding this comment

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

If this is a dependency for nemo, we can put it together to installers.

Copy link
Author

Choose a reason for hiding this comment

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

1551750786799_ pic
I built a versa conda environment from scratch, and when I run the test test/testpipeline/test_general.py I encounter this issue. It seems to be a dependency inside espnet2, but it doesn’t appear to be listed in the setup.py at https://github.com/ftshijt/espnet/blob/espnet_inference/setup.py.

Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, I see the point. Thanks for brining it up. Sure, it should be fine then (I probably will clean it up later for this hhh)

@ftshijt
Copy link
Contributor

ftshijt commented Jun 24, 2025

Looks great! I will merge it after the CI test.

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