search

ufal / treex

Treex NLP framework
33 stars 6 forks source link

Fix for a very rare crash of FixAtree #24

Closed tuetschek closed 8 years ago

tuetschek commented 8 years ago

For the sentence "WHich BBC do you watch?" (occurs in WMT15 News discussions), the tagger and parser results look like this:

    WHich   tag=IN  parent=watch
    BBC     tag=NNP parent=WHich
    ...

The previous implementation would cause a cycle in that case. This fixes the cycle, but does not attempt to fix the messed up POS tagging.

martinpopel commented 8 years ago

Thanks.

tuetschek commented 8 years ago

@martinpopel : Did you really mean to close the commit without merging?

martinpopel commented 8 years ago

Oh sorry, I though I am merging it jointly with thanking in a comment.