[PIO-34]Update templates.yaml

[bansarishah]

Integrate new predictionIO template of sentiment analysis - Bag of words model.

[dszeto]

Hi @bansarishah , thanks for the pull request. Would you mind changing the subject to include the JIRA ticket number so that they are linked? It would be sufficient to just add [PIO-34] in front of the current subject.

[bansarishah]

Hi..
I have changed subject from "Update templates.yaml " to "[PIO-34] Update
templates.yaml"
Please consider it

Thank you

Regards,
Bansari Shah

On Thu, Sep 8, 2016 at 5:24 AM, Donald Szeto notifications@github.com
wrote:

Hi @bansarishah https://github.com/bansarishah , thanks for the pull
request. Would you mind changing the subject to include the JIRA ticket
number so that they are linked? It would be sufficient to just add
[PIO-34] in front of the current subject.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#292 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ATJryECMVXvkAan7STHUBFM-n-ZzbLZUks5qn06ygaJpZM4J1tIG
.

[dszeto]

Thanks for your contribution @bansarishah ! Most of it looks great to me. A few suggested improvement:

• highlight the need to change appId in engine.json
• or even better, use the PEventStore API so that it relies on the app name rather than ID

I will hold this until the Apache release is out. PredictionIO 0.10.0 has reached RC status. Once the final release is published, we will accept this PR after the engine template is modified to use the final released artifact.

[bansarishah]

I have updated template with 'PEventStore' API. Now it works on appName instead appId and have mentioned it's use in README.md
Please consider it.
Thank you

[bansarishah]

I have done one more commit with addition of new template "Topic labelling with Wikipedia" in templates.yaml. It relates to Jira Issue [PIO-34] .
Please consider it.
Thank you.

[dszeto]

Hi @bansarishah , now that 0.10.0-incubating has released, would you mind updating your template to use those new artifacts? Once that's done I will merge this in.

You might also want to redo this against the livedoc branch if you want your entry to show up immediately on the live web site. Thanks!

[bansarishah]

Hi @dszeto , I have updated template to use predictionio new release 0.10.0-incubating.
To merge with livedoc branch i have done pull request "[PIO-34]Update templates.yaml #315"
Please consider it.
Thank you

[dszeto]

Thank you @bansarishah ! Merging this to livedoc.