search

coli-saar / am-parser

Modular implementation of an AM dependency parser in AllenNLP.
Apache License 2.0
30 stars 10 forks source link

Tops in ucca #57

Closed namednil closed 4 years ago

namednil commented 4 years ago

Currently, the post-processing doesn't take into account the root (top) node of the evaluated graph to turn into (heuristically?) into the top of the UCCA graph.

namednil commented 4 years ago

The implemented version has extremely high recall and the precision is ok, is there still something easy that could be done or shall we close the issued, @mariomgmn ?

alexanderkoller commented 4 years ago

Let's deprioritize.

namednil commented 4 years ago

Even though it is deprioritized, is it to be expected that some of our parser outputs have two top nodes? @mariomgmn

namednil commented 4 years ago

We said it's easy, and it's doable in post-processing.

alexanderkoller commented 4 years ago

Fixed at very short notice.