LexPredict / lexpredict-lexnlp

LexNLP by LexPredict
GNU Affero General Public License v3.0
690 stars 175 forks source link

get_companies not working #25

Open janlukasschroeder opened 5 years ago

janlukasschroeder commented 5 years ago

Two problems:

  1. running the example in your docs returns faulty results
import lexnlp.extract.en.entities.nltk_re
text = "This is Deutsche Bank Securities Inc."
print(list(lexnlp.extract.en.entities.nltk_re.get_companies(text)))
# [('This is Deutsche Bank Securities', 'Inc', 'Bank')]

I didn't expect "This is" to be included in the result. That's of no value at.

  1. running the example with different input doesn't work at all
import lexnlp.extract.en.entities.nltk_re
text = "Google acquired Tableau"
print(list(lexnlp.extract.en.entities.nltk_re.get_companies(text)))
# [ ]

The result is empty. I expected to see Google and Tableau.

Link to faulty example: https://lexpredict-lexnlp.readthedocs.io/en/latest/modules/extract/en/companies.html#extract-en-companies

ericlex commented 5 years ago

Thanks for sending this. We will need look into this further. Do you or your company have a support contract with us?

themarisolhernandez commented 2 years ago

Are there any updates for this in version 2.2.1.0?