search

mozilla / firefox-translations-training

Training pipelines for Firefox Translations neural machine translation models
https://mozilla.github.io/firefox-translations-training/
Mozilla Public License 2.0
135 stars 28 forks source link

Fix data importing for sacrebleu #658

Closed gregtatum closed 3 weeks ago

gregtatum commented 4 weeks ago

Resolves #74. The directionality of the language pairs is not specified in sacrebleu, and it can fail when we get it wrong. [skip ci]

gregtatum commented 4 weeks ago

I've validated this fix here: https://firefox-ci-tc.services.mozilla.com/tasks/groups/PzLlsmuDQ7eMfm3pxYgxDA#sacrebleu

I'd like to merge this in release, and we can move fixes back into main. I used this push to kick off cs-en again.