-
Notifications
You must be signed in to change notification settings - Fork 27
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
Layout engine may introduce some diffs #25
Comments
Note that in general, I think a rule of thumb is that in coreference or predicate conjunction or gapping, a variable is expanded where it is mentioned explicitly, and appears as a reentrancy where a pronoun is used or the argument is elided. So in the case of
It makes sense to expand the variable As another example from the guidelines, in
I don't know how easy it is to take these issues into account, though. |
Thanks for explaining. Those are good guidelines for hand-annotation, but I don't think it would help for serializing from triples since we don't know the surface form. It may be possible to use the alignments, if available, and the As an aside, for the |
I'm "arguing" for the original annotation, actually. In |
Oh, my mistake. I was misreading the graph. Thanks for pointing that out. |
One goal of the project is to model the PENMAN structure as graphs but to retain enough information from their serialization so the tree structure doesn't change on reserialization. Here is an example from the Bio-AMR corpus where a diff is introduced:
Here is what is produced (with whitespace differences normalized):
Note how the reentrancy of the
p
node is reversed. The layout engine prefers edges to appear in their original orientation, but in this case they do. I could possibly prefer reentrancies to start from deeper nestings, or maybe I could embed some info about reentrancy in the triple (as I do with inversion).The text was updated successfully, but these errors were encountered: