search

brendano / stanford_corenlp_pywrapper

151 stars 59 forks source link

Modifications #2

Closed johnb30 closed 10 years ago

johnb30 commented 10 years ago

Major points:

All of this works the way we need it to for the PETRARCH processing pipeline. I haven't tested it extensively for other use cases.

brendano commented 10 years ago

awesome! i'll tweak it a tiny bit after this

brendano commented 10 years ago

hey question: why did you write this codebase "requires" the shift-reduce parser? I just thought that sample.ini happens to use it. It should just run with whatever the corenlp defaults are if the config file is not set, right? i'll test in a few moments...

johnb30 commented 10 years ago

Yea. You’re right. That was just sloppy language on my part.

John Beieler johnbeieler.org

On Tuesday, July 29, 2014 at 2:32 PM, brendano wrote:

hey question: why did you write this codebase "requires" the shift-reduce parser? I just thought that sample.ini happens to use it. It should just run with whatever the corenlp defaults are if the config file is not set, right? i'll test in a few moments...

— Reply to this email directly or view it on GitHub (https://github.com/brendano/stanford-corepywrapper/pull/2#issuecomment-50518434).