Wordseer / stanford-corenlp-python

Python wrapper for Stanford CoreNLP tools
GNU General Public License v2.0
58 stars 27 forks source link

Support for CoreNLP 3.4 #1

Open Liontooth opened 10 years ago

Liontooth commented 10 years ago

Greetings! It looks like you have the most up-to-date version of this package -- any chance you could update it to CoreNLP 3.4?

When I just change the version numbers in corenlp.py, I get a hang:

# python corenlp/corenlp.py
java -Xmx3g -cp stanford-corenlp-full-2014-06-16/stanford-corenlp-3.4.jar:stanford-corenlp-full-2014-06-16/stanford-corenlp-3.4-models.jar:stanford-corenlp-full-2014-06-16/xom.jar:stanford-corenlp-full-2014-06-16/joda-time.jar:stanford-corenlp-full-2014-06-16/jollyday.jar edu.stanford.nlp.pipeline.StanfordCoreNLP -props /mnt/tvspare/software/python/stanford-corenlp-python/stanford-corenlp-python-master/corenlp/default.properties
Loading Models: 2/5

Cheers, Dave

abendebury commented 10 years ago

Yep, currently it does not support the most recent version of core NLP.

Right now development efforts are more focused on our wordseer repository, but someday we may implement support for newer versions.

Perhaps one of these links will be helpful.

https://github.com/brendano/stanford_corenlp_pywrapper

https://github.com/johnb30/stanford_corenlp_pywrapper

http://nlp.stanford.edu/software/corenlp.shtml#Extensions

Liontooth commented 10 years ago

OK, thanks. Looking forward to the WordSeer release! Are the changes in CoreNLP from a year ago so extensive that it's quite a bit of work to update your tools? None of the dasmith forks support 3.4 yet -- sorry I take that back; zigeuner forked your version on July 16 and promptly added support for CoreNLP 3.4 -- details.

Cheers, Dave

abendebury commented 10 years ago

Great to hear! To be honest, we're not really sure how much work it would be to update this tool. We're really trying to put the finishing touches on wordseer right now, and that's just been a much higher priority than updating what version of corenlp we're using.

That link looks like a 404 to me?

Liontooth commented 10 years ago

I understand. The project looks impressive. Fixed the link!

abendebury commented 10 years ago

I'll keep this issue open, hopefully one day we can update this package.