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

Use syntax information #66

Open
alexanderkoller opened this issue Jul 22, 2019 · 2 comments
Open

Use syntax information #66

alexanderkoller opened this issue Jul 22, 2019 · 2 comments
Assignees
Labels
amr eds mrp2019 Issues related to the MRP 2019 shared task nice-to-have Not essential ucca

Comments

@alexanderkoller
Copy link
Contributor

One thing we could do that might help us across the board is to use syntax information from the companion data. The easiest thing to do would be to pass the label of the incoming dependency edge of each word to the neural network.

@namednil, how much work would this be? I suppose we would simply add another column to amconll and pass the syntax edge label there?

@alexanderkoller alexanderkoller added mrp2019 Issues related to the MRP 2019 shared task amr eds ucca labels Jul 22, 2019
@namednil
Copy link
Contributor

namednil commented Jul 22, 2019

Sure, we could do that, or, even use the relative POS-based encoding.

That's not much work, an hour I guess (spent in am-tools and in the parser), and we don't even have to re-decompose everything.

@alexanderkoller
Copy link
Contributor Author

Sounds great! Let's try it!

@namednil namednil self-assigned this Jul 22, 2019
@namednil namednil added the nice-to-have Not essential label Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
amr eds mrp2019 Issues related to the MRP 2019 shared task nice-to-have Not essential ucca
Projects
None yet
Development

No branches or pull requests

2 participants