Skip to content
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

Merged
merged 17 commits into from
Aug 12, 2021

Conversation

Palipoor
Copy link
Contributor

A commonsense reasoning task based on the Com2Sense dataset.
https://arxiv.org/abs/2106.00969

@swarooprm
Copy link
Contributor

Please rename your task to task116 since task115 is already submitted in the latest Pull request.

@Palipoor Palipoor changed the title Task 113 Com2sense task - Commonsense reasoning Task 116 Com2sense task - Commonsense reasoning Jul 29, 2021
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
Copy link
Contributor

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".

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That makes sense, thanks.

Copy link
Contributor

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'.",
Copy link
Contributor

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".

Copy link
Contributor

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"
Copy link
Contributor

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"
Copy link
Contributor

Choose a reason for hiding this comment

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

Add Classification, too.

@danyaljj
Copy link
Contributor

That's all the thoughts I had.
After addressing them, @swarooprm feel free to merge when you're happy.

@danyaljj
Copy link
Contributor

danyaljj commented Aug 4, 2021

@Palipoor pinging the thread to make sure you have seen the comments above.

@Palipoor
Copy link
Contributor Author

Palipoor commented Aug 4, 2021

@danyaljj Yes I have seen them, I was just too busy these days. Will handle them soon.

Copy link
Contributor

@danyaljj danyaljj left a 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. ✅

@swarooprm
Copy link
Contributor

Looks good! Merging this task.
However, I think you can create more tasks from the same dataset. Section 2.2. contain multiple steps in the crowdsourcing process, you can convert some of those to tasks. To clarify, any input-output can constitute a task. A task does not necessarily have to be an end task in a dataset. Feel free to create new PR if you find any new task using this dataset.

@swarooprm swarooprm merged commit 03c7fdd into allenai:master Aug 12, 2021
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