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
Fix merge_label utility argument order #133
Conversation
This makes it more understandable
@@ -34,7 +34,6 @@ models: \ | |||
lvwiki_models \ | |||
nlwiki_models \ | |||
nowiki_models \ | |||
nowiki_models \ |
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.
I don't know how it got there but it's redundant and the auto-generation removed it, same for other removals.
Codecov Report
@@ Coverage Diff @@
## master #133 +/- ##
=======================================
Coverage 14.08% 14.08%
=======================================
Files 56 56
Lines 987 987
=======================================
Hits 139 139
Misses 848 848 Continue to review full report at Codecov.
|
Makefile
Outdated
@@ -550,7 +546,7 @@ datasets/enwiki.human_labeled_revisions.20k_2015.json: | |||
https://labels.wmflabs.org/campaigns/enwiki/4/ > $@ | |||
|
|||
datasets/enwiki.labeled_revisions.20k_2015.json: \ | |||
datasets/enwiki.human_labeled_revisions.20k_2015.json | |||
datasets/enwiki.human_labeled_revisions20k_2015.json |
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.
Missing a "."
templates/Makefile.j2
Outdated
datasets/{{ wiki.name }}.{{ sample }}.{{ sample_name }}.json{% if not loop.last %} \{% endif -%} | ||
{% endfor %} | ||
{%- if big_sample.human_labeled_revisions %} | ||
datasets/{{ wiki.name }}.human_labeled_revisions{{ big_sample.human_labeled_revisions }}.json{%- if big_sample.autolabeled_revisions %} \ |
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.
I think you need to add the "." here
6ae40b1
to
49b4ffa
Compare
Done, so stupid (facepalm) |
Bug: https://phabricator.wikimedia.org/T187742