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.

Sign up for GitHub

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

Jump to bottom

Minor improvements #20

Merged
merged 15 commits into from
Aug 17, 2021
Merged

Minor improvements #20

merged 15 commits into from
Aug 17, 2021

Conversation

eu9ene
Copy link
Collaborator

@eu9ene eu9ene commented Aug 17, 2021 •
edited
Loading

  • Flores dataset importer
  • custom dataset importer
  • ability to use a pre-trained backward model
  • save experiment config on start
  • stubs for dataset caching ( decided to sync implementation with workflow manager integration )
  • use best bleu models instead of best ce-mean-words
  • fix linting warnings

fixes #17 #12 #14 #11

@eu9ene eu9ene requested review from andrenatal and abhi-agg August 17, 2021 15:45
@eu9ene eu9ene merged commit 0f6e64c into mozilla:main Aug 17, 2021
This was linked to issues Aug 25, 2021
Custom training/test dataset #11
Closed
Add ability to use custom backward model #12
Closed
Track experiments metadata #14
Closed
nordzilla pushed a commit that referenced this pull request Sep 19, 2024
@kpu
Merge pull request #20 from browsermt/andrenatal-patch-1
f43dc33 
Update README.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andrenatal andrenatal andrenatal approved these changes

@abhi-agg abhi-agg Awaiting requested review from abhi-agg

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@eu9ene @andrenatal