-
Notifications
You must be signed in to change notification settings - Fork 812
re-parameterization of Multi30K dataset #1207
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1207 +/- ##
==========================================
+ Coverage 73.21% 73.23% +0.02%
==========================================
Files 67 67
Lines 3689 3718 +29
==========================================
+ Hits 2701 2723 +22
- Misses 988 995 +7
Continue to review full report at Codecov.
|
MD5 = {'train': [], 'valid': [], 'test': []} | ||
NUM_LINES = {'train': [], 'valid': [], 'test': []} | ||
|
||
for task in SUPPORTED_DATASETS.keys(): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the "in" operator should return a key by default, so I don't think you need .keys() here
Summary: