Re: Update to OpenNLP 1.7.1

2017-03-08 Thread James Masanz
Sorry Jörn I had forgotten you had wanted to commit the patch yourself. I backed out the commit I made. On Thu, Jan 26, 2017 at 5:14 AM, Joern Kottmann wrote: > Yes, the version should be updated to 1.7.1 and the opennlp maxent > dependency should be removed. > I prepared a patch yesterday and w

Re: Update to OpenNLP 1.7.1

2017-01-26 Thread Joern Kottmann
mailto:kottm...@gmail.com]  > Sent: Thursday, January 26, 2017 5:15 AM > To: dev@ctakes.apache.org > Subject: Re: Update to OpenNLP 1.7.1 > > Yes, the version should be updated to 1.7.1 and the opennlp maxent > dependency should be removed. > I prepared a patch yesterday and will open a j

RE: Update to OpenNLP 1.7.1

2017-01-26 Thread Finan, Sean
Tonic! -Original Message- From: Joern Kottmann [mailto:kottm...@gmail.com] Sent: Thursday, January 26, 2017 5:15 AM To: dev@ctakes.apache.org Subject: Re: Update to OpenNLP 1.7.1 Yes, the version should be updated to 1.7.1 and the opennlp maxent dependency should be removed. I prepared

Re: Update to OpenNLP 1.7.1

2017-01-26 Thread Joern Kottmann
Yes, the version should be updated to 1.7.1 and the opennlp maxent dependency should be removed. I prepared a patch yesterday and will open a jira tonight and attach it there. Let me attach the patch, then I will ask for a review and if you don't mind I would then commit it myself. Jörn On Thu,

Re: Update to OpenNLP 1.7.1

2017-01-25 Thread Andrey Kurdumov
Hello Joern, What changes needed to update to OpenNLP 1.7.1? Show I change only pom.xml or something else? I would like to try it out. I maintain some kind of cTaks fork, exactly for purpose of keeping up with dependency updates, until they land in the Ctakes itself. 2017-01-26 4:00 GMT+06:00 Joe

Re: Update to OpenNLP 1.7.1

2017-01-25 Thread Joern Kottmann
Hello, yes I did a lot of testing to make sure that 1.7.0 and 1.7.1 are as good as any other OpenNLP release. The OpenNLP eval tests use all the data we currently support to ensure it all works. If you have some specific corpora you need to be sure OpenNLP performs well on we can try to add them

RE: Update to OpenNLP 1.7.1

2017-01-25 Thread Finan, Sean
Hi Jorn, Great idea! Have you done any testing with the latest release? I think that ctakes has 3 dependencies on opennlp libraries. We should try to get our integration with that update (and others) solidified for the upcoming release. The boston children's hospital nlp team just gained a