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
-
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