-
Notifications
You must be signed in to change notification settings - Fork 190
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
Task103 facts to story #34
Conversation
Thanks for the PR! Few very minor suggestions:
Long text generation:
Please update the readme of task names as well. |
I updated the task 102 file - Sentence Generation. Please review. I have a question regarding Task 103 - Long text generation: |
Sorry, I miss-typed. There are a bunch of non-natural( |
Got it. I updated both the files according to your suggestions. Please review. Thanks! |
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.
Looks good to me! Just a minor suggestion:
Change the file names that indicate the source dataset and the category e.g. if source for both tasks are 'commongen', then task name should be taskNNN
I renamed the file and uploaded the final ones. |
Accept from my side! |
Added new task 103: Facts to Story. Please review.