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

Rouge score #157

Open
Muennighoff opened this issue May 25, 2023 · 1 comment
Open

Rouge score #157

Muennighoff opened this issue May 25, 2023 · 1 comment

Comments

@Muennighoff
Copy link

It says Multi-lingual ROUGE is unsupported as general token splitting is absent from [rouge-score](https://github.com/google-research/google-research/tree/master/rouge). For multi-lingual tasks, please ignore rouge metrics until this is resolved. NOTE: English works as intended., but it also works for e.g. Spanish and other languages that split on space like English, right?

cc @jon-tow

@jon-tow
Copy link
Collaborator

jon-tow commented May 25, 2023

I probably would not recommend it for Spanish or any other "normal" spaced lang in the current state. The default tokenizer used in rouge_scorer replaces non-alphanumeric chars (English) with spaces, so, for example, the text "Cristóbal está ayudando a su Abuela" would be converted to "Cristbal est ayudando a su Abuela"; removing the ó and á. See the tokenize definition here:
https://github.com/google-research/google-research/blob/0aa035ff363066089612fb37e3e137a71cadb9c0/rouge/tokenize.py#L50-L61
Though, if you could loosen the non_alpha_numeric pattern to ignore accented letters etc. it should be fine.

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

No branches or pull requests

2 participants