[GitHub] [opennlp-sandbox] jzonthemtn merged pull request #61: Update sandbox component 'mallet-addon' to be compatible with latest opennlp-tools release

[GitBox]

jzonthemtn merged PR #61:
URL: https://github.com/apache/opennlp-sandbox/pull/61


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] mawiesne commented on pull request #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 1.8.2 in /mallet-addon

[GitBox]

mawiesne commented on PR #50:
URL: https://github.com/apache/opennlp-sandbox/pull/50#issuecomment-1398272054

   Closing in favor of #61.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] dependabot[bot] commented on pull request #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 1.8.2 in /mallet-addon

[GitBox]

dependabot[bot] commented on PR #50:
URL: https://github.com/apache/opennlp-sandbox/pull/50#issuecomment-1398271746

   OK, I won't notify you again about this release, but will get in touch when 
a new version is available. If you'd rather skip all updates until the next 
major or minor version, let me know by commenting `@dependabot ignore this 
major version` or `@dependabot ignore this minor version`.
   
   If you change your mind, just re-open this PR and I'll resolve any conflicts 
on it.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] mawiesne closed pull request #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 1.8.2 in /mallet-addon

[GitBox]

mawiesne closed pull request #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 
1.8.2 in /mallet-addon
URL: https://github.com/apache/opennlp-sandbox/pull/50


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] mawiesne commented on pull request #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 1.8.2 in /mallet-addon

[GitBox]

mawiesne commented on PR #50:
URL: https://github.com/apache/opennlp-sandbox/pull/50#issuecomment-1398189559

   This PR can be closed, provided 
https://github.com/apache/opennlp-sandbox/pull/61 is merged.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] mawiesne opened a new pull request, #61: Updates sandbox component 'mallet-addon' to be compatible with latest opennlp-tools release

[GitBox]

mawiesne opened a new pull request, #61:
URL: https://github.com/apache/opennlp-sandbox/pull/61

   - adjusts opennlp-tools to 2.1.0
   - adjusts parent project (org.apache.apache) to version 18
   - adjusts Java language level to 11
   - updates to mallet version 2.0.8 to mitigate several CVEs, adds exclusions 
and related newer versions to mitigate CVEs
   - adjusts some array declarations to comply with Java, not C, style
   - improves resource handling of streams
   - removes unused imports


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [opennlp-sandbox] dependabot[bot] opened a new pull request, #50: Bump opennlp-tools from 1.6.1-SNAPSHOT to 1.8.2 in /mallet-addon

[GitBox]

dependabot[bot] opened a new pull request, #50:
URL: https://github.com/apache/opennlp-sandbox/pull/50

   Bumps [opennlp-tools](https://github.com/apache/opennlp) from 1.6.1-SNAPSHOT 
to 1.8.2.
   
   Commits
   
   See full diff in https://github.com/apache/opennlp/commits/opennlp-1.8.2;>compare 
view
   
   
   
   
   
   [![Dependabot compatibility 
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=org.apache.opennlp:opennlp-tools=maven=1.6.1-SNAPSHOT=1.8.2)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)
   
   Dependabot will resolve any conflicts with this PR as long as you don't 
alter it yourself. You can also trigger a rebase manually by commenting 
`@dependabot rebase`.
   
   [//]: # (dependabot-automerge-start)
   [//]: # (dependabot-automerge-end)
   
   ---
   
   
   Dependabot commands and options
   
   
   You can trigger Dependabot actions by commenting on this PR:
   - `@dependabot rebase` will rebase this PR
   - `@dependabot recreate` will recreate this PR, overwriting any edits that 
have been made to it
   - `@dependabot merge` will merge this PR after your CI passes on it
   - `@dependabot squash and merge` will squash and merge this PR after your CI 
passes on it
   - `@dependabot cancel merge` will cancel a previously requested merge and 
block automerging
   - `@dependabot reopen` will reopen this PR if it is closed
   - `@dependabot close` will close this PR and stop Dependabot recreating it. 
You can achieve the same result by closing it manually
   - `@dependabot ignore this major version` will close this PR and stop 
Dependabot creating any more for this major version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this minor version` will close this PR and stop 
Dependabot creating any more for this minor version (unless you reopen the PR 
or upgrade to it yourself)
   - `@dependabot ignore this dependency` will close this PR and stop 
Dependabot creating any more for this dependency (unless you reopen the PR or 
upgrade to it yourself)
   - `@dependabot use these labels` will set the current labels as the default 
for future PRs for this repo and language
   - `@dependabot use these reviewers` will set the current reviewers as the 
default for future PRs for this repo and language
   - `@dependabot use these assignees` will set the current assignees as the 
default for future PRs for this repo and language
   - `@dependabot use this milestone` will set the current milestone as the 
default for future PRs for this repo and language
   
   You can disable automated security fix PRs for this repo from the [Security 
Alerts page](https://github.com/apache/opennlp-sandbox/network/alerts).
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@opennlp.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Re: mallet addon

[Rodrigo Agerri]

Hello,

Thanks. I thought I had an idea for CRF not obtaining good results
with OpenNLP default features, e.g.,

http://lingpipe-blog.com/2006/11/22/why-do-you-hate-crfs/

but if results are also worse in Maxent, that is intriguing. I will
look at the Mallet implementation to see if I find out something.

R



On Mon, Oct 12, 2015 at 4:07 PM, Joern Kottmann  wrote:
> Hello,
>
> fixed up the code a bit. The performance is not really good. Do you have
> any idea why that could be?
>
> Neither the maxent or crf get good evaluation numbers on NER.
>
> I will push the changes and then you can experiment with it too.
>
> Jörn
>
>
> On Mon, Oct 5, 2015 at 4:45 PM, Rodrigo Agerri  wrote:
>
>> Hi,
>>
>> On Tue, Sep 29, 2015 at 3:41 PM, Joern Kottmann 
>> wrote:
>> > We can also move
>> > it to the sandbox, releasing it at Apache might be more difficult since
>> > mallet pulls in incompatible licensed dependencies. But maybe that
>> changed,
>> > we can check.
>>
>> Mallet is released under Common Public License
>>
>> http://opensource.org/licenses/cpl1.0.php
>>
>> but as you have mentioned, it pulls several dependencies that are
>> LGPL. These are the dependencies:
>>
>>   
>>   org.beanshell
>>   bsh
>>   2.0b4
>> 
>>
>> This version is LGPL, however, later versions are APL 2.0
>>
>> https://github.com/beanshell/beanshell
>>
>> 
>>   jgrapht
>>   jgrapht
>>   0.6.0
>> 
>>
>> that version was also LGPL, but it has now been dual-licensed with EPL 1.0
>>
>> https://github.com/jgrapht/jgrapht/wiki/Relicensing
>>
>> which could be included also in APL 2.0 projects
>>
>> http://www.apache.org/legal/resolved.html
>>
>>  
>>   net.sf.jwordnet
>>   jwnl
>>   1.4_rc3
>> 
>>
>> BSD license, but this library has already been discussed here.
>>
>>  
>>   net.sf.trove4j
>>   trove4j
>>   2.0.2
>> 
>>
>> LGPL-ed.
>>
>> 
>>   com.googlecode.matrix-toolkits-java
>>   mtj
>>   0.9.14
>> 
>>
>> also LGPL
>>
>> Rodrigo
>>

Re: mallet addon

[Joern Kottmann]

Hello,

I updated the code and afterwards spent some time evaluating it again. The
maxent training is very close to our maxent classifier. I also checked the
training code again and it looks good to me, but it would be nice if you
can review it.

There are a couple of other classifiers in mallet, it should be trivial to
expose them all to OpenNLP.

Jörn

On Tue, Oct 20, 2015 at 9:12 AM, Rodrigo Agerri 
wrote:

> Hello,
>
> Thanks. I thought I had an idea for CRF not obtaining good results
> with OpenNLP default features, e.g.,
>
> http://lingpipe-blog.com/2006/11/22/why-do-you-hate-crfs/
>
> but if results are also worse in Maxent, that is intriguing. I will
> look at the Mallet implementation to see if I find out something.
>
> R
>
>
>
> On Mon, Oct 12, 2015 at 4:07 PM, Joern Kottmann 
> wrote:
> > Hello,
> >
> > fixed up the code a bit. The performance is not really good. Do you have
> > any idea why that could be?
> >
> > Neither the maxent or crf get good evaluation numbers on NER.
> >
> > I will push the changes and then you can experiment with it too.
> >
> > Jörn
> >
> >
> > On Mon, Oct 5, 2015 at 4:45 PM, Rodrigo Agerri 
> wrote:
> >
> >> Hi,
> >>
> >> On Tue, Sep 29, 2015 at 3:41 PM, Joern Kottmann 
> >> wrote:
> >> > We can also move
> >> > it to the sandbox, releasing it at Apache might be more difficult
> since
> >> > mallet pulls in incompatible licensed dependencies. But maybe that
> >> changed,
> >> > we can check.
> >>
> >> Mallet is released under Common Public License
> >>
> >> http://opensource.org/licenses/cpl1.0.php
> >>
> >> but as you have mentioned, it pulls several dependencies that are
> >> LGPL. These are the dependencies:
> >>
> >>   
> >>   org.beanshell
> >>   bsh
> >>   2.0b4
> >> 
> >>
> >> This version is LGPL, however, later versions are APL 2.0
> >>
> >> https://github.com/beanshell/beanshell
> >>
> >> 
> >>   jgrapht
> >>   jgrapht
> >>   0.6.0
> >> 
> >>
> >> that version was also LGPL, but it has now been dual-licensed with EPL
> 1.0
> >>
> >> https://github.com/jgrapht/jgrapht/wiki/Relicensing
> >>
> >> which could be included also in APL 2.0 projects
> >>
> >> http://www.apache.org/legal/resolved.html
> >>
> >>  
> >>   net.sf.jwordnet
> >>   jwnl
> >>   1.4_rc3
> >> 
> >>
> >> BSD license, but this library has already been discussed here.
> >>
> >>  
> >>   net.sf.trove4j
> >>   trove4j
> >>   2.0.2
> >> 
> >>
> >> LGPL-ed.
> >>
> >> 
> >>   com.googlecode.matrix-toolkits-java
> >>   mtj
> >>   0.9.14
> >> 
> >>
> >> also LGPL
> >>
> >> Rodrigo
> >>
>

Re: mallet addon

[Eldad Yamin]

How can I unsubscribe?
On Sep 29, 2015 15:35, "Rodrigo Agerri" <rage...@apache.org> wrote:

> Hello,
>
> I have seen that there is a mallet addon here
>
> https://github.com/kottmann/opennlp-mallet-addon
>
> is this currently being used or integrated in opennlp? I have not seen
> with the rest of the addons.
>
> Cheers,
>
> Rodrigo
>

Re: mallet addon

[Russ, Daniel (NIH/CIT) [E]]

from:  https://opennlp.apache.org/mail-lists.html
To un-subscribe send an e-mail to 
dev-unsubscr...@opennlp.apache.org<mailto:dev-unsubscr...@opennlp.apache.org>


Dan


On Oct 20, 2015, at 10:43 AM, Eldad Yamin 
<elda...@gmail.com<mailto:elda...@gmail.com>> wrote:

How can I unsubscribe?
On Sep 29, 2015 15:35, "Rodrigo Agerri" 
<rage...@apache.org<mailto:rage...@apache.org>> wrote:

Hello,

I have seen that there is a mallet addon here

https://github.com/kottmann/opennlp-mallet-addon

is this currently being used or integrated in opennlp? I have not seen
with the rest of the addons.

Cheers,

Rodrigo

Re: mallet addon

[Rodrigo Agerri]

Hi,

On Tue, Sep 29, 2015 at 3:41 PM, Joern Kottmann  wrote:
> We can also move
> it to the sandbox, releasing it at Apache might be more difficult since
> mallet pulls in incompatible licensed dependencies. But maybe that changed,
> we can check.

Mallet is released under Common Public License

http://opensource.org/licenses/cpl1.0.php

but as you have mentioned, it pulls several dependencies that are
LGPL. These are the dependencies:

  
  org.beanshell
  bsh
  2.0b4


This version is LGPL, however, later versions are APL 2.0

https://github.com/beanshell/beanshell


  jgrapht
  jgrapht
  0.6.0


that version was also LGPL, but it has now been dual-licensed with EPL 1.0

https://github.com/jgrapht/jgrapht/wiki/Relicensing

which could be included also in APL 2.0 projects

http://www.apache.org/legal/resolved.html

 
  net.sf.jwordnet
  jwnl
  1.4_rc3


BSD license, but this library has already been discussed here.

 
  net.sf.trove4j
  trove4j
  2.0.2


LGPL-ed.


  com.googlecode.matrix-toolkits-java
  mtj
  0.9.14


also LGPL

Rodrigo

Re: mallet addon

[Rodrigo Agerri]

I think moving it to the sandbox will be very nice. I can also help if
needed.  I will check the mallet license.

Thanks

Rodrigo
On 29 Sep 2015 15:41, "Joern Kottmann" <kottm...@gmail.com> wrote:

> Hello,
>
> this doesn't work with the 1.6.0 release, I build it for testing of one of
> the first drafts of the machine learning rewrite work we did for 1.6.0.
> There have been a few changes afterwards.
> Anyway, if you have a need for it I am happy to fix it up. We can also move
> it to the sandbox, releasing it at Apache might be more difficult since
> mallet pulls in incompatible licensed dependencies. But maybe that changed,
> we can check.
>
> What do you think?
>
> Jörn
>
>
>
> On Tue, Sep 29, 2015 at 2:34 PM, Rodrigo Agerri <rage...@apache.org>
> wrote:
>
> > Hello,
> >
> > I have seen that there is a mallet addon here
> >
> > https://github.com/kottmann/opennlp-mallet-addon
> >
> > is this currently being used or integrated in opennlp? I have not seen
> > with the rest of the addons.
> >
> > Cheers,
> >
> > Rodrigo
> >
>

mallet addon

[Rodrigo Agerri]

Hello,

I have seen that there is a mallet addon here

https://github.com/kottmann/opennlp-mallet-addon

is this currently being used or integrated in opennlp? I have not seen
with the rest of the addons.

Cheers,

Rodrigo

Re: mallet addon

[Joern Kottmann]

Hello,

this doesn't work with the 1.6.0 release, I build it for testing of one of
the first drafts of the machine learning rewrite work we did for 1.6.0.
There have been a few changes afterwards.
Anyway, if you have a need for it I am happy to fix it up. We can also move
it to the sandbox, releasing it at Apache might be more difficult since
mallet pulls in incompatible licensed dependencies. But maybe that changed,
we can check.

What do you think?

Jörn



On Tue, Sep 29, 2015 at 2:34 PM, Rodrigo Agerri <rage...@apache.org> wrote:

> Hello,
>
> I have seen that there is a mallet addon here
>
> https://github.com/kottmann/opennlp-mallet-addon
>
> is this currently being used or integrated in opennlp? I have not seen
> with the rest of the addons.
>
> Cheers,
>
> Rodrigo
>