Skip to content

spBELU-1K - #282

Merged
martinpopel merged 3 commits into
mjpost:masterfrom
elmadany:master
Aug 30, 2025
Merged

martinpopel merged 3 commits into
mjpost:masterfrom
elmadany:master

Conversation

@elmadany

Copy link
Copy Markdown
Contributor

spBLEU-1K is a newly introduced multilingual SentencePiece tokenizer, trained on over 1,000 data sources, as proposed in the Toucan paper (ACL 2024). For more details, see spBLEU's GitHub repository.

spBLEU-1K is a newly introduced multilingual SentencePiece tokenizer, trained on over 1,000 data sources, as proposed in the Toucan paper (ACL 2024)
@martinpopel
martinpopel merged commit a542538 into mjpost:master Aug 30, 2025
19 checks passed
@martinpopel

Copy link
Copy Markdown
Collaborator

Thank you for your contribution. While I don't think we should include too many different tokenizers into sacrebleu (simple whitespace+punctuation tokenization is enough for all languages that separate words with spaces), having an alternative SPM model "covering 1,003 languages, including 614 African languages, 53 Indigenous American languages" seems like a good idea.

@elmadany

elmadany commented Sep 2, 2025

Copy link
Copy Markdown
Contributor Author

Thanks, @martinpopel, for merging the spBLEU-1k! I plan to announce this soon.

I was wondering when you plan to release the new version on PyPI. It would be great if users could simply run pip install sacrebleu instead of having to install it directly from the git repository (pip install git+https://github.com/mjpost/sacrebleu.git).

@martinpopel

Copy link
Copy Markdown
Collaborator

@mjpost would you want to release 2.5.2? (I have never done it, I think, but it seems I have the rights at PyPI, so I could try).

Comment thread README.md
- `ko-mecab` tokenizes **Korean** inputs using the [MeCab-ko](https://pypi.org/project/mecab-ko) morphological analyzer
- `flores101` and `flores200` uses the SentencePiece model built from the Flores-101 and [Flores-200](https://github.com/facebookresearch/flores/blob/main/flores200/README.md#languages-in-flores-200) dataset, respectively. Note: the canonical .spm file will be automatically fetched if not found locally.
- 🔥🆕 `spBLEU-1K` is a newly introduced **multilingual** SentencePiece tokenizer, trained on **over 1,000** data sources, as proposed in the [Toucan](https://aclanthology.org/2024.findings-acl.781/) paper (ACL 2024). For more details, see spBLEU's [GitHub repository](https://github.com/UBC-NLP/Toucan/tree/main/spBLEU-1K).

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Can you remove these two blank lines?

Comment thread sacrebleu/metrics/bleu.py
'spm': 'tokenizer_spm.TokenizerSPM',
'flores101': 'tokenizer_spm.Flores101Tokenizer',
'flores200': 'tokenizer_spm.Flores200Tokenizer',
### Added for spBLEU-1K tokenizer by AbdelRahim Elmadany

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

I suggest you link to the paper: https://aclanthology.org/2024.findings-acl.781/

"spBLEU-1K": {
"url": "https://www.dlnlp.ai/spBLEU-1K/spbleu-1k_tokenizer_spm.model",
"signature": "spBLEU-1K",
},

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please fix the indentation.

@mjpost

mjpost commented Sep 3, 2025

Copy link
Copy Markdown
Owner

@martinpopel Sure, that's fine. If I remember, you can actually publish it by creating a tag. Then when you push, it is uploaded automatically. I always forget how to do this, though.

@mjpost

mjpost commented Sep 3, 2025

Copy link
Copy Markdown
Owner

We need to update the CHANGELOG, too.

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.

3 participants