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=expand tokenize=None (uses 13a default tokenizer) while insacrebleupython lib (https://github.com/mjpost/sacrebleu), the default params seem to besmooth_method=floorandtokenize='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