You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@opennlp.apache.org by "Russ, Daniel (NIH/CIT) [E]" <dr...@mail.nih.gov> on 2015/05/28 15:34:52 UTC
GIS API
Hi,
The interface for opennlp.tools.maxent.GIS is changing between 1.5 and 1.6. This will cause me to have to make many changes in my code that could be easily avoided if we change make opennlp.tools.ml.model extend ObjectStream<Event>. Also, Since EventStream is not used by GIS, is there any reason why it is not deprecated? I will put this in JIRA.
Thank you
Dan
Re: GIS API
Posted by Joern Kottmann <ko...@gmail.com>.
Looks like there are a couple of classes depending on it which can be all
removed. Lets clean that up for 1.6.0.
Jörn
On Thu, May 28, 2015 at 3:51 PM, Joern Kottmann <ko...@gmail.com> wrote:
> Yes, or we can also remove it directly. The package name changed anyway,
> we will not be able to support existing code using EventStream with 1.6.0
> without breaking it.
>
> Please open a jia for it.
>
> Jörn
>
> On Thu, May 28, 2015 at 3:49 PM, Russ, Daniel (NIH/CIT) [E] <
> druss@mail.nih.gov> wrote:
>
>> Sorry, I just saw the interfaces are very different, so there is not easy
>> fix. EventStream still should be deprecated.
>> Dan
>>
>>
>> On May 28, 2015, at 9:34 AM, Russ, Daniel (NIH/CIT) [E] wrote:
>>
>> > Hi,
>> >
>> > The interface for opennlp.tools.maxent.GIS is changing between 1.5
>> and 1.6. This will cause me to have to make many changes in my code that
>> could be easily avoided if we change make opennlp.tools.ml.model extend
>> ObjectStream<Event>. Also, Since EventStream is not used by GIS, is there
>> any reason why it is not deprecated? I will put this in JIRA.
>> >
>> > Thank you
>> > Dan
>>
>>
>
Re: GIS API
Posted by Joern Kottmann <ko...@gmail.com>.
Yes, or we can also remove it directly. The package name changed anyway, we
will not be able to support existing code using EventStream with 1.6.0
without breaking it.
Please open a jia for it.
Jörn
On Thu, May 28, 2015 at 3:49 PM, Russ, Daniel (NIH/CIT) [E] <
druss@mail.nih.gov> wrote:
> Sorry, I just saw the interfaces are very different, so there is not easy
> fix. EventStream still should be deprecated.
> Dan
>
>
> On May 28, 2015, at 9:34 AM, Russ, Daniel (NIH/CIT) [E] wrote:
>
> > Hi,
> >
> > The interface for opennlp.tools.maxent.GIS is changing between 1.5 and
> 1.6. This will cause me to have to make many changes in my code that could
> be easily avoided if we change make opennlp.tools.ml.model extend
> ObjectStream<Event>. Also, Since EventStream is not used by GIS, is there
> any reason why it is not deprecated? I will put this in JIRA.
> >
> > Thank you
> > Dan
>
>
Re: GIS API
Posted by "Russ, Daniel (NIH/CIT) [E]" <dr...@mail.nih.gov>.
Sorry, I just saw the interfaces are very different, so there is not easy fix. EventStream still should be deprecated.
Dan
On May 28, 2015, at 9:34 AM, Russ, Daniel (NIH/CIT) [E] wrote:
> Hi,
>
> The interface for opennlp.tools.maxent.GIS is changing between 1.5 and 1.6. This will cause me to have to make many changes in my code that could be easily avoided if we change make opennlp.tools.ml.model extend ObjectStream<Event>. Also, Since EventStream is not used by GIS, is there any reason why it is not deprecated? I will put this in JIRA.
>
> Thank you
> Dan