Return-Path: X-Original-To: apmail-manifoldcf-dev-archive@www.apache.org Delivered-To: apmail-manifoldcf-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id A5D3117D54 for ; Tue, 26 Jan 2016 15:49:37 +0000 (UTC) Received: (qmail 51487 invoked by uid 500); 26 Jan 2016 15:49:37 -0000 Delivered-To: apmail-manifoldcf-dev-archive@manifoldcf.apache.org Received: (qmail 51416 invoked by uid 500); 26 Jan 2016 15:49:37 -0000 Mailing-List: contact dev-help@manifoldcf.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@manifoldcf.apache.org Delivered-To: mailing list dev@manifoldcf.apache.org Received: (qmail 51148 invoked by uid 99); 26 Jan 2016 15:49:37 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 26 Jan 2016 15:49:37 +0000 Received: from mail-lf0-f45.google.com (mail-lf0-f45.google.com [209.85.215.45]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id 585171A003F for ; Tue, 26 Jan 2016 15:49:36 +0000 (UTC) Received: by mail-lf0-f45.google.com with SMTP id c192so107617341lfe.2 for ; Tue, 26 Jan 2016 07:49:36 -0800 (PST) X-Gm-Message-State: AG10YOQlT7XPu+HlBOHaybanchA99Rmxi2w0/lWlSAF+BAYu3JayvwyTfybDu6S7Nuemm7CDFcUIw2zqsBwK2w== X-Received: by 10.25.18.231 with SMTP id 100mr9060951lfs.25.1453823374966; Tue, 26 Jan 2016 07:49:34 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Rafa Haro Date: Tue, 26 Jan 2016 15:49:25 +0000 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: Chalitha's OpenNLP Transformation Connector Contribution To: dev@manifoldcf.apache.org Cc: chalithaudara@gmail.com Content-Type: multipart/alternative; boundary=001a113f67eed4c7aa052a3ea1be --001a113f67eed4c7aa052a3ea1be Content-Type: text/plain; charset=UTF-8 Hi all, I have imported Chalitha's OpenNLP connector to a branch: You can checkout it here: https://svn.apache.org/repos/asf/manifoldcf/branches/CONNECTORS-1270 License files and ant build are still lacking Thanks a lot Chalitha for the contribution. On Tue, Jan 26, 2016 at 1:48 PM Rafa Haro wrote: > Ok, > > For this I would need Chalitha to fork the current Manifold's repo in > github, put the connector in the correct place in his fork and made the > pull request. > > @Chalitha, if you are available to do this in a short period of time, it > would be nice. If you are currently busy I can take care or just go through > the classical "manual" process > > Cheers, > Rafa > > On Tue, Jan 26, 2016 at 1:38 PM Karl Wright wrote: > >> Hi Rafa, >> >> I am fine with that approach if you want to try it. >> >> Karl >> >> >> On Tue, Jan 26, 2016 at 7:34 AM, Rafa Haro wrote: >> >> > Hi Chalitha, >> > >> > I have created a branch for contributing the connector but probably you >> > should first merge your develop branch into the master branch because >> it is >> > better for me to work against the master branch. >> > >> > @karl: I'm ready for importing this at SVN level (I already have >> locally a >> > OpenNlpTransformationConnector branch) but, would not be better to >> receive >> > a pull request at GitHub from Chalitha now that we know how to quickly >> get >> > a Git pull request imported to the SVN repo? >> > >> > Cheers, >> > Rafa >> > >> > On Tue, Jan 26, 2016 at 12:07 PM Karl Wright >> wrote: >> > >> > > Any news on this? >> > > Karl >> > > >> > > >> > > On Fri, Jan 22, 2016 at 7:26 AM, Karl Wright >> wrote: >> > > >> > > > " I think I can proceed to create a branch for this and start the >> > > > merge process. What do you think?" >> > > > >> > > > I agree. Once you have a branch, please let me know and I'll see >> about >> > > > licensing of dependencies etc. Will you be integrating the ant >> build? >> > > > >> > > > Karl >> > > > >> > > > >> > > > On Fri, Jan 22, 2016 at 7:10 AM, Rafa Haro >> wrote: >> > > > >> > > >> Hi Chalitha, >> > > >> >> > > >> I have reviewed the code and my opinion right now is quite ok for a >> > > first >> > > >> release. I still have to build it and check the UI, but from the >> code >> > > >> seems >> > > >> to be enough for configuring the models paths. Could you provide a >> > > >> screenshot of the configuration? >> > > >> >> > > >> @Karl, I think I can proceed to create a branch for this and start >> the >> > > >> merge process. What do you think? >> > > >> >> > > >> Cheers, >> > > >> Rafa >> > > >> >> > > >> On Mon, Jan 18, 2016 at 4:01 PM Rafa Haro >> wrote: >> > > >> >> > > >> > Hi Chalitha, >> > > >> > I will review it this week and let you know something here. >> > > >> > >> > > >> > Cheers, >> > > >> > Rafa >> > > >> > >> > > >> > On Sat, Jan 16, 2016 at 11:20 AM chalitha udara Perera < >> > > >> > chalithaudara@gmail.com> wrote: >> > > >> > >> > > >> >> Hi Rafa and Karl, >> > > >> >> >> > > >> >> I have externalise the model configuration and now user can >> provide >> > > >> model >> > > >> >> location from Job configuration UI. >> > > >> >> Connector code is avaliable in [1]. >> > > >> >> Please have a look when you have some time. >> > > >> >> >> > > >> >> [1] >> > > >> >> >> > > >> >> > > >> https://github.com/ChalithaUdara/OpenNLP-Manifold-Connector/tree/develop >> > > >> >> >> > > >> >> Thanks, >> > > >> >> Chalitha >> > > >> >> >> > > >> >> On Sun, Dec 20, 2015 at 11:23 AM, chalitha udara Perera < >> > > >> >> chalithaudara@gmail.com> wrote: >> > > >> >> >> > > >> >> > Hi Rafa and Karl, >> > > >> >> > >> > > >> >> > To give more flexibility, It make sense to externalize models >> and >> > > go >> > > >> >> with >> > > >> >> > configuration information. >> > > >> >> > I'll modify the connector, so that users can provide models. >> > > >> >> > >> > > >> >> > Thanks, >> > > >> >> > Chalitha >> > > >> >> > >> > > >> >> > On Fri, Dec 18, 2015 at 5:30 PM, Karl Wright < >> daddywri@gmail.com >> > > >> > > >> >> wrote: >> > > >> >> > >> > > >> >> >> If the models are immutable resources, then we should >> probably >> > > just >> > > >> >> bundle >> > > >> >> >> them in the jar as resources. If they are modifiable and/or >> it >> > is >> > > >> >> >> expected >> > > >> >> >> that the user will supply their own, then configuration >> > > information >> > > >> is >> > > >> >> the >> > > >> >> >> way to go. >> > > >> >> >> >> > > >> >> >> Karl >> > > >> >> >> >> > > >> >> >> >> > > >> >> >> On Fri, Dec 18, 2015 at 6:58 AM, Karl Wright < >> > daddywri@gmail.com> >> > > >> >> wrote: >> > > >> >> >> >> > > >> >> >> > Hi Rafa, >> > > >> >> >> > >> > > >> >> >> > I'm also not really happy with how the connector locates >> the >> > > >> models. >> > > >> >> >> It's >> > > >> >> >> > not going to work very well in a distributed environment. >> It >> > > >> seems >> > > >> >> like >> > > >> >> >> > there's a missing piece here: the models should probably be >> > > >> imported >> > > >> >> as >> > > >> >> >> > part of the connection's configuration information, rather >> > than >> > > >> just >> > > >> >> an >> > > >> >> >> > implicit file reference. How big are these model files, >> > > >> typically? >> > > >> >> >> > >> > > >> >> >> > Karl >> > > >> >> >> > >> > > >> >> >> > >> > > >> >> >> > On Fri, Dec 18, 2015 at 6:53 AM, Karl Wright < >> > > daddywri@gmail.com> >> > > >> >> >> wrote: >> > > >> >> >> > >> > > >> >> >> >> Yes, please create a ticket. But I'd create a branch and >> > > import >> > > >> the >> > > >> >> >> code >> > > >> >> >> >> into that first. I have to get licenses and build process >> > > >> working >> > > >> >> >> before >> > > >> >> >> >> we can bring everything into trunk. >> > > >> >> >> >> >> > > >> >> >> >> Thanks, >> > > >> >> >> >> Karl >> > > >> >> >> >> >> > > >> >> >> >> >> > > >> >> >> >> On Fri, Dec 18, 2015 at 6:38 AM, Rafa Haro < >> rharo@apache.org >> > > >> > > >> >> wrote: >> > > >> >> >> >> >> > > >> >> >> >>> Hi all, >> > > >> >> >> >>> >> > > >> >> >> >>> I was about to open a Jira ticket for including >> Chalitha's >> > > >> OpenNLP >> > > >> >> >> >>> Transformation Connector ( >> > > >> >> >> >>> >> https://github.com/ChalithaUdara/OpenNLP-Manifold-Connector >> > ) >> > > >> into >> > > >> >> the >> > > >> >> >> >>> trunk. >> > > >> >> >> >>> >> > > >> >> >> >>> Everyone agree? >> > > >> >> >> >>> >> > > >> >> >> >>> Cheers, >> > > >> >> >> >>> Rafa >> > > >> >> >> >>> >> > > >> >> >> >> >> > > >> >> >> >> >> > > >> >> >> > >> > > >> >> >> >> > > >> >> > >> > > >> >> > >> > > >> >> > >> > > >> >> > -- >> > > >> >> > J.M Chalitha Udara Perera >> > > >> >> > >> > > >> >> > *Department of Computer Science and Engineering,* >> > > >> >> > *University of Moratuwa,* >> > > >> >> > *Sri Lanka* >> > > >> >> > >> > > >> >> >> > > >> >> >> > > >> >> >> > > >> >> -- >> > > >> >> J.M Chalitha Udara Perera >> > > >> >> >> > > >> >> *Department of Computer Science and Engineering,* >> > > >> >> *University of Moratuwa,* >> > > >> >> *Sri Lanka* >> > > >> >> >> > > >> > >> > > >> >> > > > >> > > > >> > > >> > >> > --001a113f67eed4c7aa052a3ea1be--