-
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
Task 116 Com2sense task - Commonsense reasoning #60
Conversation
Please rename your task to task116 since task115 is already submitted in the latest Pull request. |
tasks/README.md
Outdated
`task108_ContextualAbuseDetection_classification` | Given a text detect whether it's abusive or not. | Classification | ||
`task_116_com2sense_commonsense_reasoning` | Decide whether a sentence is true or not, based on commonsense | Commonsense Reasoning |
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.
task_116
-> task116
I would suggest changing "sentence is true" to whether "the provided event is plausible".
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.
That makes sense, 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.
For the last column, maybe also add Classification
, too.
@@ -0,0 +1,12051 @@ | |||
{ | |||
"Definition": "You will be given one sentence about an everyday event. You need to decide whether the sentence is true or false, based on your commonsense. For true sentences output 'True' and for false sentences output 'False'.", |
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 am not sure what it means for a sentence to be "true" or "false".
I would suggest changing the phrasing of the instruction, to something along the line of, "you need to decide whether provided story in the following sentence is a plausible scenario and it matches your commonsense".
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.
Similarly, instead of saying "For true sentences output 'True'" (which sounds like a recursive statement), say, "if the given event is a plausible one, output 'True'" and so on.
} | ||
], | ||
"Source": [ | ||
"Com2Sense" |
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.
Expand this a bit, something like this: The instructions and the instance are extracted from the Com2Sense ()
"Pegah Alipoormolabashi" | ||
], | ||
"Categories": [ | ||
"Commonsense Reasoning" |
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.
Add Classification
, too.
That's all the thoughts I had. |
@Palipoor pinging the thread to make sure you have seen the comments above. |
@danyaljj Yes I have seen them, I was just too busy these days. Will handle them soon. |
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! This is good to go, from my perspective. ✅
Looks good! Merging this task. |
A commonsense reasoning task based on the Com2Sense dataset.
https://arxiv.org/abs/2106.00969