Skip to content

Adds Fever NLI task and data downloader#1215

Merged
jeswan merged 9 commits intonyu-mll:masterfrom
angie-chen55:fever_nli
Nov 5, 2020
Merged

Adds Fever NLI task and data downloader#1215
jeswan merged 9 commits intonyu-mll:masterfrom
angie-chen55:fever_nli

Conversation

@angie-chen55
Copy link
Member

No description provided.

@pep8speaks
Copy link

Hello @angie-chen55! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 161:101: E501 line too long (105 > 100 characters)
Line 170:101: E501 line too long (105 > 100 characters)

You can repair most issues by installing black and running: black -l 100 ./*. If you contribute often, have a look at the 'Contributing' section of the README for instructions on doing this automatically.

Copy link
Collaborator

@jeswan jeswan left a comment

Choose a reason for hiding this comment

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

Thanks for this PR! Please add step 4 of https://github.com/nyu-mll/jiant/blob/master/guides/tasks/adding_tasks.md and address the formatting error.

@angie-chen55 angie-chen55 marked this pull request as ready for review October 26, 2020 22:52
@angie-chen55 angie-chen55 requested a review from jeswan October 26, 2020 22:53
@jeswan jeswan merged commit e7eefc6 into nyu-mll:master Nov 5, 2020
leo-liuzy pushed a commit to leo-liuzy/dynamic_jiant that referenced this pull request Nov 11, 2020
* Implement task and add downloader for Fever NLI

Co-authored-by: jeswan <57466294+jeswan@users.noreply.github.com>
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