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

Bugs/issues listed at googlecode #50

Closed
linas opened this issue Dec 27, 2014 · 6 comments
Closed

Bugs/issues listed at googlecode #50

linas opened this issue Dec 27, 2014 · 6 comments

Comments

@linas
Copy link
Member

linas commented Dec 27, 2014

There are currently eleven open issues listed at googlecode:

https://code.google.com/p/link-parser/issues/list?cursor=link-parser%3A58&ts=1419717174

A cut-n-paste shows:

10  Defect      Accepted    Medium  ----     Construction: "[Verb] like [phrase].    
14  Defect      Accepted    Medium  ----     Questions involving comparatives    
15  Defect      Accepted    Medium  ----     "If ..., then ..." fails to parse   
27  Defect      Accepted    Medium  ----     incomplete constituent tree     
28  Defect      Accepted    Medium  ----    ldouble dashes are mis-parsed    
31  Enhancement     Assigned    Medium  ----    add support for circumpositions      
34  Enhancement     Assigned    Low     ----    Elegent normal form      
50  Defect      New     Medium  ----    to pattern: "I have to reading books." is accepted   
58  Enhancement     Started     High    ----    Head word links needed.      
62  Defect      Accepted    Medium  ----    Russian capitalization broken    
63  Defect      Accepted    Medium  ----    Imperatives, commands broken 
@ampli
Copy link
Member

ampli commented Jan 8, 2016

What is already fix and what still need fixing?
The full list (needed in order to see the details of each issue) is now at https://code.google.com/archive/p/link-parser/issues . Everything is marked now as fixed,Is this correct? At least 28 has never been fixed.

This is maybe mostly fine by now:
62 Defect Accepted Medium ---- Russian capitalization broken
(But it is not yet clear to me why names appear in parses both as UC and LC - dict definition problem?)

This still needs a fix:
28 Defect Accepted Medium ---- ldouble dashes are mis-parsed
Edit: It got fixed in PR #575.

@linas
Copy link
Member Author

linas commented Jan 8, 2016

I think that marking them as "fixed" was a way to indicate I'd copied them to here. Seems like a bad idea.

@linas
Copy link
Member Author

linas commented Jan 8, 2016

Working from memory:
comparatives: I think many but not all got fixed. Likewise for imperatives. The head-word stuff is mostly done.

The "incomplete constituent tree" is probably a bug deep in the constituent code, which I have no desire to fiddle with.

@linas
Copy link
Member Author

linas commented Jun 17, 2017

10 58 and 63 are all fixed.

@ampli
Copy link
Member

ampli commented Nov 7, 2019

Update:
28 got fixed in PR #575.

@linas
Copy link
Member Author

linas commented Nov 7, 2019

I'm going to close this as "stale". 34 - elegant normal form will never happen. 27 needs an example. The others are English dict issues, on which there has been much progress, and also clearly, plenty of existing examples of sentences that don't parse right.

@linas linas closed this as completed Nov 7, 2019
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