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

invalid MRS? #25

Closed
arademaker opened this issue Sep 25, 2020 · 1 comment
Closed

invalid MRS? #25

arademaker opened this issue Sep 25, 2020 · 1 comment

Comments

@arademaker
Copy link
Member

Trunk version with sentence communicate or express by writing

[ TOP: h0
  INDEX: e2 [ e SF: prop TENSE: tensed MOOD: indicative PROG: - PERF: - ]
  RELS: < [ _communicate_v_to<0:11> LBL: h1 ARG0: e4 [ e SF: prop TENSE: tensed MOOD: indicative PROG: - PERF: - ] ARG1: i3 ARG2: h5 ARG3: i6 ]
          [ _or_c<12:14> LBL: h1 ARG0: e2 ARG1: e4 ARG2: e7 [ e SF: prop TENSE: pres MOOD: indicative PROG: - PERF: - ] ]
          [ _express_v_to<15:22> LBL: h1 ARG0: e7 ARG1: i3 ARG2: h8 ARG3: i9 ]
          [ unknown<23:33> LBL: h10 ARG: u12 ARG0: e11 [ e SF: prop TENSE: untensed MOOD: indicative ] ]
          [ _by_p_means<23:25> LBL: h10 ARG0: e11 ARG1: u13 ARG2: x14 ]
          [ udef_q<26:33> LBL: h15 ARG0: x14 RSTR: h16 BODY: h17 ]
          [ nominalization<26:33> LBL: h18 ARG0: x14 ARG1: h19 ]
          [ _write_v_to<26:33> LBL: h19 ARG0: e20 [ e SF: prop TENSE: untensed MOOD: indicative PROG: + PERF: - ] ARG1: i21 ARG2: i22 ] >
  HCONS: < h0 qeq h1 h5 qeq h23 h8 qeq h23 h16 qeq h18 > ]
@danflick
Copy link
Collaborator

danflick commented Apr 2, 2021

This dictionary definition phrase with missing objects for the two verbs should only be parsable using the variant of the ERG intended for parsing dictionary definitions. I fixed a bug in the right-node-raising rule that was wrongly analyzing this example with the standard ERG, and will check in the change to `trunk' shortly. To compile the variant grammar, do ace -G erg-dict.dat -g ace/config-dict.tdl and you can confirm that the top-ranked parse of the above has the correct MRS.

@danflick danflick closed this as completed Apr 2, 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

No branches or pull requests

2 participants