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 qeqs cause DMRS/EDS conversion to raise KeyError #303

Closed
goodmami opened this issue Jul 17, 2020 · 1 comment
Closed

Invalid qeqs cause DMRS/EDS conversion to raise KeyError #303

goodmami opened this issue Jul 17, 2020 · 1 comment
Labels
Milestone

Comments

@goodmami
Copy link
Member

goodmami commented Jul 17, 2020

Currently a bug in MRS-to-DMRS (or EDS) conversion causes a KeyError when the lo handle of a qeq is not the label of some EP. This sometimes happens when the TOP handle h0 is qeq to h1 but h1 is not the label of anything. In the following example (item 1150 from the ERG's handp12 profile), the qeq from the scopal argument of neg is not the LBL of any EP:

[ TOP: h0
  INDEX: e2 [ e SF: prop TENSE: past MOOD: indicative ]
  RELS: < [ neg<3:9> LBL: h1 ARG0: e4 [ e SF: prop TENSE: untensed MOOD: indicative PROG: - PERF: - ] ARG1: h5 ]
          [ _for_p<10:13> LBL: h1 ARG0: e6 [ e SF: prop TENSE: untensed MOOD: indicative ] ARG1: u7 ARG2: x8 [ x PERS: 1 NUM: sg IND: + PT: std ] ]
          [ pron<14:16> LBL: h9 ARG0: x8 ]
          [ pronoun_q<14:16> LBL: h10 ARG0: x8 RSTR: h11 BODY: h12 ]
          [ loc_nonsp<17:21> LBL: h1 ARG0: i13 ARG1: e2 ARG2: e6 ]
          [ pron<22:24> LBL: h14 ARG0: x15 [ x PERS: 3 NUM: sg GEND: m IND: + PT: std ] ]
          [ pronoun_q<22:24> LBL: h16 ARG0: x15 RSTR: h17 BODY: h18 ]
          [ _make_v_1<25:29> LBL: h1 ARG0: e2 ARG1: x15 ARG2: x19 [ x PERS: 3 NUM: sg PT: pt ] ]
          [ _the_q<30:33> LBL: h20 ARG0: x19 RSTR: h21 BODY: h22 ]
          [ _sacrifice_n_1<34:43> LBL: h23 ARG0: x19 ] >
  HCONS: < h0 qeq h1 h5 qeq h24 h11 qeq h9 h17 qeq h14 h21 qeq h23 >
  ICONS: < e2 focus e6 > ]

While these are bad MRSs and there might not be a reasonable way to recover from these errors, the error message could be more useful to the user. And processes like delphin convert should be able to detect the error and move on instead of stopping altogether.

@goodmami goodmami added the bug label Jul 17, 2020
@goodmami
Copy link
Member Author

A reasonable solution may be to issue a warning and otherwise ignore it (that is, set DMRS.top to None or drop the link). This way one might still be able to get something useful out of the (possibly disconnected) DMRS, a conversion process wouldn't terminate mid-way through, and the user can still be aware of the problem from the warning.

@goodmami goodmami added this to the v1.4.0 milestone Jul 20, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant