default param values for sacrebleu #4289
Replies: 1 comment
-
Hi! The default parameters should match since our implementation calls |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
I see default params for sacrebleu as
smooth_method=exp
and tokenize=None (uses 13a default tokenizer) while insacrebleu
python lib (https://github.com/mjpost/sacrebleu), the default params seem to besmooth_method=floor
andtokenize='none'
(which uses BaseTokenizer).Which set is officially recognized and why was exp smooth method chosen in
hf datasets
?Beta Was this translation helpful? Give feedback.
All reactions