Hi Chanaka,

Thanks for the review. I'll do the necessary changes accordingly.

I'll add the required unit tests as well.

Thanks.

On Wed, Apr 1, 2015 at 10:25 AM, Chanaka Fernando <chana...@wso2.com> wrote:

> Hi Manorama,
>
> I have added few comments to the PR. Please review them and do the
> necessary changes to the PR. Is it possible to write some unit tests for
> the mediator?
>
> Thanks,
> Chanaka
>
> On Wed, Apr 1, 2015 at 10:03 AM, Manorama Perera <manor...@wso2.com>
> wrote:
>
>> Hi Nirmal,
>>
>> I've sent the PR[1] to carbon-mediation repo
>>
>> [1] https://github.com/wso2/carbon-mediation/pull/100
>>
>> Thanks.
>>
>> On Wed, Apr 1, 2015 at 6:53 AM, Nirmal Fernando <nir...@wso2.com> wrote:
>>
>>> [Removing architecture@ and moving the discussion to dev@]
>>>
>>> Hi Manorama,
>>>
>>> Shall we work with the ESB team and get this mediator merged to the
>>> relevant repo please?
>>>
>>> On Wed, Mar 25, 2015 at 3:35 PM, Nirmal Fernando <nir...@wso2.com>
>>> wrote:
>>>
>>>> Thanks everyone.
>>>>
>>>> So, we decided to have following improvements;
>>>>
>>>> 1. /s/ml/machineLearner
>>>> 2. Instead of specifying storage-location, we should get the model-name
>>>> from the user.
>>>> 3. feature elements should be wrapped from a features element.
>>>> 4. prediction element currently replaces an existing element in the
>>>> incoming message, we can support the append as well.
>>>>
>>>> On Tue, Mar 24, 2015 at 11:56 PM, Chanaka Fernando <chana...@wso2.com>
>>>> wrote:
>>>>
>>>>> Hi Manorama,
>>>>>
>>>>> I guess that the names are not finalized yet. The name "ml" does not
>>>>> make much impression on the users. Please give a valid name like
>>>>> "MachineLearner" and that would make life easier for ESB developers. All
>>>>> our mediators has self explanatory names.
>>>>>
>>>>> Thanks,
>>>>> Chanaka
>>>>>
>>>>> On Tue, Mar 24, 2015 at 4:09 PM, Supun Sethunga <sup...@wso2.com>
>>>>> wrote:
>>>>>
>>>>>> Hi Maninda,
>>>>>>
>>>>>> With the initial release of ML, the model would not update on the
>>>>>> fly. If we need to update the model, it has to be done in ML by training 
>>>>>> a
>>>>>> model with the same configurations, but with the new data. In future
>>>>>> releases we will be looking at this for possible improvements.
>>>>>>
>>>>>> Here the term "domain" refers to the definition of the data which use
>>>>>> to train the model. In the case you mentioned, If the model was trained
>>>>>> using the data collected from all incoming requests; then domain is "all
>>>>>> sales". and can be used to handle all incoming requests. But if the model
>>>>>> was trained using the data collected from the incoming requests of, say
>>>>>> Seller_A, then it cannot be used to predict/handle incoming requests of 
>>>>>> all
>>>>>> sellers, which is a different domain.
>>>>>> When we say "domain", it also includes the Schema (set of
>>>>>> features/variables) of the dataset on which the prediction is going to be
>>>>>> done. Handling multiple schema is tricky. If the data from a different
>>>>>> domain doesn't contain some of the features (variables in the dataset) 
>>>>>> used
>>>>>> to train the model, then that model cannot be used to predict on that
>>>>>> particular data. So we have to work with this limitation, hence all the
>>>>>> data has to be on the same domain (i.e. should have the same schema).
>>>>>>
>>>>>> Regards,
>>>>>> Supun
>>>>>>
>>>>>> On Tue, Mar 24, 2015 at 3:10 PM, Maninda Edirisooriya <
>>>>>> mani...@wso2.com> wrote:
>>>>>>
>>>>>>> Does this update the model time to time or use the same modal
>>>>>>> throughout the ESB up time? It is better if we can update the model 
>>>>>>> while
>>>>>>> the ESB is up and running.
>>>>>>> There may be some use cases related to multiple domains right? So I
>>>>>>> think would be more generic and useful if we can apply the mediator not
>>>>>>> based on the domain but with a custom argument expression. For example 
>>>>>>> if
>>>>>>> the target service is SalesService and the operation is 
>>>>>>> getTotalSalesCount
>>>>>>> we want to use the modal to analyze all incoming requests from all 
>>>>>>> domains.
>>>>>>> WDYT?
>>>>>>>
>>>>>>> Thanks.
>>>>>>>
>>>>>>>
>>>>>>> *Maninda Edirisooriya*
>>>>>>> Senior Software Engineer
>>>>>>>
>>>>>>> *WSO2, Inc.*lean.enterprise.middleware.
>>>>>>>
>>>>>>> *Blog* : http://maninda.blogspot.com/
>>>>>>> *E-mail* : mani...@wso2.com
>>>>>>> *Skype* : @manindae
>>>>>>> *Twitter* : @maninda
>>>>>>>
>>>>>>> On Tue, Mar 24, 2015 at 12:00 PM, Manorama Perera <manor...@wso2.com
>>>>>>> > wrote:
>>>>>>>
>>>>>>>> Hi,
>>>>>>>>
>>>>>>>> ESB mediator for WSO2 ML enables making use of the models generated
>>>>>>>> by WSO2 Machine Leaner, to predict values in message mediation in ESB.
>>>>>>>>
>>>>>>>>    - Machine Learner feature needs to be installed in ESB prior to
>>>>>>>>    configure the ML mediator.
>>>>>>>>    - Mediator will extract the data from the incoming messages and
>>>>>>>>    send to the ML model as features.
>>>>>>>>    - The mediator use the Machine Learner features to predict the
>>>>>>>>    output value for every message passing through the mediator.
>>>>>>>>
>>>>>>>> *Proposed Mediator Configuration*
>>>>>>>>
>>>>>>>> <ml>
>>>>>>>>     <model storage-location="string"/>
>>>>>>>>     <feature name="string" expression="xpath"/>+
>>>>>>>>     <prediction expression="xpath"/>
>>>>>>>> </ml>
>>>>>>>>
>>>>>>>> *model*
>>>>>>>> storage-location : Storage location of the generated model.
>>>>>>>>
>>>>>>>> *feature*
>>>>>>>> name : Feature name according to the generated model.
>>>>>>>> expression : XPath expression used to determine the feature value.
>>>>>>>>
>>>>>>>> *prediction*
>>>>>>>> expression : XPath expression to determine the location to insert
>>>>>>>> the predicted value.
>>>>>>>>
>>>>>>>> The generated models can be applied to predict only the data coming
>>>>>>>> from the same domain, as the data used to train the model. So the 
>>>>>>>> mediator
>>>>>>>> should be configured only to mediate messages which carry data coming 
>>>>>>>> from
>>>>>>>> the same domain, as the data used to train the model.
>>>>>>>>
>>>>>>>> Any suggestion are appreciated.
>>>>>>>>
>>>>>>>> Thanks.
>>>>>>>>
>>>>>>>> --
>>>>>>>> Manorama Perera
>>>>>>>> Software Engineer
>>>>>>>> WSO2, Inc.;  http://wso2.com/
>>>>>>>> Mobile : +94716436216
>>>>>>>>
>>>>>>>> _______________________________________________
>>>>>>>> Architecture mailing list
>>>>>>>> architect...@wso2.org
>>>>>>>> https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> _______________________________________________
>>>>>>> Architecture mailing list
>>>>>>> architect...@wso2.org
>>>>>>> https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>> --
>>>>>> *Supun Sethunga*
>>>>>> Software Engineer
>>>>>> WSO2, Inc.
>>>>>> http://wso2.com/
>>>>>> lean | enterprise | middleware
>>>>>> Mobile : +94 716546324
>>>>>>
>>>>>> _______________________________________________
>>>>>> Architecture mailing list
>>>>>> architect...@wso2.org
>>>>>> https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>> --
>>>>> --
>>>>> Chanaka Fernando
>>>>> Technical Lead
>>>>> WSO2, Inc.; http://wso2.com
>>>>> lean.enterprise.middleware
>>>>>
>>>>> mobile: +94 773337238
>>>>> Blog : http://soatutorials.blogspot.com
>>>>> LinkedIn:http://www.linkedin.com/pub/chanaka-fernando/19/a20/5b0
>>>>> Twitter:https://twitter.com/chanakaudaya
>>>>> Wordpress:http://chanakaudaya.wordpress.com
>>>>>
>>>>>
>>>>>
>>>>>
>>>>> _______________________________________________
>>>>> Architecture mailing list
>>>>> architect...@wso2.org
>>>>> https://mail.wso2.org/cgi-bin/mailman/listinfo/architecture
>>>>>
>>>>>
>>>>
>>>>
>>>> --
>>>>
>>>> Thanks & regards,
>>>> Nirmal
>>>>
>>>> Senior Software Engineer- Platform Technologies Team, WSO2 Inc.
>>>> Mobile: +94715779733
>>>> Blog: http://nirmalfdo.blogspot.com/
>>>>
>>>>
>>>>
>>>
>>>
>>> --
>>>
>>> Thanks & regards,
>>> Nirmal
>>>
>>> Senior Software Engineer- Platform Technologies Team, WSO2 Inc.
>>> Mobile: +94715779733
>>> Blog: http://nirmalfdo.blogspot.com/
>>>
>>>
>>>
>>
>>
>> --
>> Manorama Perera
>> Software Engineer
>> WSO2, Inc.;  http://wso2.com/
>> Mobile : +94716436216
>>
>
>
>
> --
> --
> Chanaka Fernando
> Technical Lead
> WSO2, Inc.; http://wso2.com
> lean.enterprise.middleware
>
> mobile: +94 773337238
> Blog : http://soatutorials.blogspot.com
> LinkedIn:http://www.linkedin.com/pub/chanaka-fernando/19/a20/5b0
> Twitter:https://twitter.com/chanakaudaya
> Wordpress:http://chanakaudaya.wordpress.com
>
>
>
>


-- 
Manorama Perera
Software Engineer
WSO2, Inc.;  http://wso2.com/
Mobile : +94716436216
_______________________________________________
Dev mailing list
Dev@wso2.org
http://wso2.org/cgi-bin/mailman/listinfo/dev

Reply via email to