explosion/spaCy

calling sents() on empty string

nartz opened this issue · 2 comments

nartz commented

I guess it was unexpected to get this error:
[s for s in nlp(u'').sents] -> IndexError: list index out of range

I.e. calling sents on an empty string errors out - I wonder if this is the right behavior, or if sents should return an empty array?

syllog1sm commented

Thanks.

This should either return an empty list or yield out of an empty generator (off the top of my head, I don't remember whether .sents is a generator...Which is probably a bad sign. =/)

  • 👍1
lock commented

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.