-
Notifications
You must be signed in to change notification settings - Fork 242
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
Modify underlying downloads and add more download materials #197
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.
LGTM!
— Could we move the bibtex entry to the bottom of the page please.
— Let's add a note at the top of the page about dataset update. Maybe something like: NOTE: Data files for all tasks / splits were updated on Feb 10, 2021 due to minor bugs (affecting < 1% of the data) in earlier versions. If you downloaded data before Feb 10, please re-download the data.
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.
LGTM - added data download updated numbers
Looks good. I agree with Abhishek that we should include some details on the update to the dataset. Probably good to include a version history. |
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.
Let's add a version history to the dataset.md. Don't include links to the old datasets, but briefly describe what changed with each version.
|
||
|
||
|
||
## Dataset changelog |
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.
@clz55 , @abhshkdz , @mshuaibii : I added this, let me know if this seems okay.
* Modify underlying downloads and add more download materials * Add checksum for mapping tables * download_data.py * Add changelog Co-authored-by: Muhammed Shuaibi <mshuaibi@andrew.cmu.edu>
This PR does the following: