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

Parser assigns head from different sentence #4688

Closed
pglock opened this issue Nov 21, 2019 · 2 comments · Fixed by #4702
Closed

Parser assigns head from different sentence #4688

pglock opened this issue Nov 21, 2019 · 2 comments · Fixed by #4702
Labels
bug Bugs and behaviour differing from documentation feat / parser Feature: Dependency Parser

Comments

@pglock
Copy link

pglock commented Nov 21, 2019

related to #3170

import de_core_news_sm

nlp = de_core_news_sm.load()
doc = nlp('''Außerdem ist Zimmer davon überzeugt, dass auch epige-
netische Mechanismen eine Rolle spielen, also Vorgänge, die
sich darauf auswirken, welche Gene abgelesen werden und
welche nicht.
''')
for sent in doc.sents:
    for token in sent:
        assert token.head in sent, (token, token.head, sent)

Info about spaCy

  • spaCy version: 2.2.2
  • Platform: Linux-5.3.11-1-MANJARO-x86_64-with-arch-Manjaro-Linux
  • Python version: 3.7.5
@adrianeboyd adrianeboyd added bug Bugs and behaviour differing from documentation feat / parser Feature: Dependency Parser labels Nov 21, 2019
@adrianeboyd
Copy link
Contributor

Thanks for the report! I can replicate this and will look into the details.

@lock
Copy link

lock bot commented Dec 26, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Dec 26, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Bugs and behaviour differing from documentation feat / parser Feature: Dependency Parser
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants