Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zulu to English #190

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Zulu to English #190

wants to merge 6 commits into from

Conversation

godide
Copy link

@godide godide commented Oct 14, 2021

Pull request benchmark after updating files, model and confirming the attached notebook.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Collaborator

@juliakreutzer juliakreutzer left a comment

Choose a reason for hiding this comment

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

Hi @godide thank you so much for your work!
Are the BLEU reports of 28.5/38.3 or those of 13.6/4.8 correct?
Please add the correct results also to https://github.com/masakhane-io/masakhane-mt/blob/master/language_pairs.md. Thanks!



## Results
- BLEU dev: 28.48
Copy link
Collaborator

Choose a reason for hiding this comment

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

These are great results. Thank you for sharing also the insights into the data processing and the analysis.

>
> 2019-11-13 07:44:03,502 dev bleu: 13.64 [Beam search decoding with beam size = 5 and alpha = 1.0]
>
> 2019-11-13 07:44:24,289 test bleu: 4.87 [Beam search decoding with beam size = 5 and alpha = 1.0]`
Copy link
Collaborator

Choose a reason for hiding this comment

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

How do these results go with the above reported ones?

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