-
Notifications
You must be signed in to change notification settings - Fork 21
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
Event definition missing in v1.0 #2
Comments
Thanks for signalling it. "rehab" should not be an event on its own. It should be merged with "checks into". Will take care in the next release of the dataset. |
There are another two issues.
Thanks! |
CASE 1:
CASE 2:
Thanks!! |
CASE 1: 1_21ecbplus.xml.xml t1_checking_into_morningside_or_showing_up t1_comply Notice that 5->7 is annotated as FALLING_ACTION but 6->7 is annotated as precondition. However, 5,6 are both t1_checking_into_morningside_or_showing_up. For more examples: 12_3ecbplus.xml.xml t12_thwarted t12_pirates_arrested CASE 2: 1_18ecbplus.xml.xml 45 92_93 t1_approved |
can you point out to examples for version 1.0 as well? version 0.9 should not be used. |
CASE 1: 1_21ecbplus.xml.xml t1_checking_into_morningside_or_showing_up t1_comply Notice that 5->7 is annotated as FALLING_ACTION but 6->7 is annotated as precondition. However, 5,6 are both t1_checking_into_morningside_or_showing_up. CASE 2: I checked the first 2 example in v0.9, they also exist in v1.0. Thanks. |
Thanks! |
In EventStoryLine/annotated_data/v1.0/1/1_11ecbplus.xml.xml,
token t_id="36" sentence="1" number="4">rehab
should be an event.
However, it is missing. In v1.0/1/1_11ecbplus.xml.xml, t_id=36 is not labeled as an event.
But in v0.9, there is:
<ACTION_OCCURRENCE m_id="35" climaxEvent="TRUE" comment="">
<token_anchor t_id="36"/>
The text was updated successfully, but these errors were encountered: