8000 Add QuaRel semantic parser by OyvindTafjord · Pull Request #1857 · allenai/allennlp · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Dec 16, 2022. It is now read-only.

Add QuaRel semantic parser #1857

Merged
merged 43 commits into from
Oct 4, 2018
Merged

Conversation

OyvindTafjord
Copy link
Collaborator

(same as #1812, just changing owner)

This adds the QuaRel semantic parser for parsing questions about qualitative relations.

Copy link
Contributor
@matt-gardner matt-gardner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Thanks for doing this @OyvindTafjord!

return args[0]
return None

def execute(self, lf_raw: str) -> int:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If this gets any more complicated, we'll probably want to move the execution logic into executors.

@OyvindTafjord OyvindTafjord merged commit 8ff8324 into allenai:master Oct 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0