-
Notifications
You must be signed in to change notification settings - Fork 8
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
Variables in ICONS do not correspond to EPs in the MRS #2
Comments
A quick comment! 吠 is used in the MRS testsuite for Zhong no longer. I Sanghoun On Mon, Jul 20, 2015 at 11:34 AM, Michael Wayne Goodman <
Sanghoun Song NTU Computational Linguistics Lab. | http://compling.hss.ntu.edu.sg |
It is still used in the master branch: Maybe this change is only in your local repository? |
Oh! This problem! I didn't update the gold-mrs yet. The version was created several months Sanghoun On Mon, Jul 20, 2015 at 11:45 AM, Michael Wayne Goodman <
Sanghoun Song NTU Computational Linguistics Lab. | http://compling.hss.ntu.edu.sg |
I found the problem. The problem comes from the combination of resultative Sanghoun On Mon, Jul 20, 2015 at 11:50 AM, Sanghoun Song [email protected] wrote:
Sanghoun Song NTU Computational Linguistics Lab. | http://compling.hss.ntu.edu.sg |
There are some sentences where variables appearing in the ICONS do not appear as the intrinsic variable (ARG0) of any EP in the MRS. Likely there is some bug in the grammar during MRS construction, e.g. where the original EP gets dropped or something.
Here are some example sentences from the
mrs
testsuite:There are many more in the MRS testsuite. I don't know if they all are caused by the same grammar bug or relate to the same grammatical phenomenon.
These variables become evident because variable properties appear inside the ICONS list. It may be ok that these properties appear here, but they only do (with, say, the ACE parser) when the variable doesn't appear anywhere else. You can search for more examples in the testsuite with this regex:
ICONS: <.*\[
(you may need to escape some things differently based on your regex processor; I used vim).The text was updated successfully, but these errors were encountered: