Skip to content

fix cmn ASR eval in SeamlessExpressive#352

Open
hygong-fb wants to merge 1 commit intomainfrom
expressive-cmn-eval
Open

fix cmn ASR eval in SeamlessExpressive#352
hygong-fb wants to merge 1 commit intomainfrom
expressive-cmn-eval

Conversation

@hygong-fb
Copy link

SeamlessExpressive evaluation of eng-cmn normalizes both transcribed and ground truth texts into simplified Chinese texts. It is observed that there is a mix of traditional and simplified Chinese characters in texts, which resulted in low ASR-BLEU. Therefore, chinese_converter is added to ASR-BLEU in SeamlessExpressive eval pipeline.

Test command:

python3 ${SEAMLESS_COMMUNICATION_ROOT}/src/seamless_communication/cli/expressivity/evaluate/run_asr_bleu.py
--generation_dir_path=${GENERATED_DIR}/${SPLIT}${SRC_LANG}${TGT_LANG}
--generate_tsv_filename=${GENERATED_TSV}
--tgt_lang=${TGT_LANG}

@hygong-fb hygong-fb requested review from kauterry and uralik February 6, 2024 00:36
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 6, 2024
Copy link
Contributor

@uralik uralik left a comment

Choose a reason for hiding this comment

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

approving, although this might also be a problem in the main m4t v2 eval

@sravyapopuri388 tagging you here for awareness

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants