-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Narrative QA Manual #1778
Narrative QA Manual #1778
Conversation
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.
Really cool thanks ! And good dataset card as well :)
I left a few comments
5f4e872
to
915330e
Compare
Excellent comments. Thanks for those valuable suggestions. I changed everything as you have pointed out :) |
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.
Thanks for moving all the processing to _generate_examples !
I was wondering whether we should use the same features format as in the narrative_qa
dataset script (same column names, same structure) for consistency. Let me know what you think
915330e
to
6ea09de
Compare
I've copied the same template as NarrativeQA now. Please let me know if this is fine. |
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.
Awesome thank you !!
This looks all good :)
Just before we merge, I was wondering if you knew why the number of examples in the train set went from 1102 to 32747 in your last commit ? I can't see why the changes in the code would cause such a big difference
Ok the change was the way I presented the data. In the case of But since this would be inconsistent as you pointed out, I modified my code to suit the |
Ok I see ! that makes sense |
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.
Thanks for adding this one :)
Thanks for your time and helping me with all this :) Really appreciate the hardwork you guys do. |
Submitting the manual version of Narrative QA script which requires a manual download from the original repository