[GitHub] incubator-predictionio issue #376: Fix docs because pio template is no longe...

2017-05-01 Thread tom-chan
Github user tom-chan commented on the issue: https://github.com/apache/incubator-predictionio/pull/376 LGTM, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] incubator-predictionio issue #327: pio build fails with 0.10.0

2016-11-11 Thread tom-chan
Github user tom-chan commented on the issue: https://github.com/apache/incubator-predictionio/issues/327 Can I have the URL for that? Things might be a little messy during transition so just want to make sure. --- If your project is set up for it, you can reply to this email

[GitHub] incubator-predictionio issue #327: pio build fails with 0.10.0

2016-11-11 Thread tom-chan
Github user tom-chan commented on the issue: https://github.com/apache/incubator-predictionio/issues/327 Where did you get the recommender from? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does

Re: [GitHub] incubator-predictionio issue #326: What's command line param for "Would you ...

2016-11-09 Thread Tom Chan
A quick fix to this is to do ``` echo y | pio template get apache/incubator-predictionio-template-recommender /opt/MyRecommendation --name Admin --package co.cococo --email supp...@cocococ.co ``` I think there's discussion about removing this check but I hope this will unblock you in the

Re: consolidate repos into 1

2016-11-03 Thread Tom Chan
This is mostly a good idea but then one of the templates is 3 times the size of incubator-predictionio: $ du -d 1 -h 53M ./incubator-predictionio 1.3M ./incubator-predictionio-sdk-java 288K ./incubator-predictionio-sdk-php 536K ./incubator-predictionio-sdk-python 264K

Re: PIO Build Error

2016-10-12 Thread Tom Chan
eturn code of previous step is 1. Aborting. > > Thanks, > > Jim > > > -- > Jim Miller > > On October 12, 2016 at 6:24:23 PM, Tom Chan (yukhei.c...@gmail.com) wrote: > > I think everything "io.prediction" in the template (imports) need to be > changed to

Re: [VOTE]: Apache PredictionIO (incubating) 0.10.0 Release

2016-09-15 Thread Tom Chan
+1 On Thu, Sep 15, 2016 at 10:23 AM, Mani Teja wrote: > +1 > > On Thu, Sep 15, 2016 at 10:35 PM, Donald Szeto wrote: > > > +1 > > > > On Thu, Sep 15, 2016 at 9:04 AM, Pat Ferrel > wrote: > > > > > Donald, you get a vote too.

[GitHub] incubator-predictionio pull request #273: PIO-20 Merge ActionML fork

2016-08-09 Thread tom-chan
Github user tom-chan commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/273#discussion_r74125867 --- Diff: conf/log4j.properties --- @@ -1,20 +1,3 @@ -# -# Licensed to the Apache Software Foundation (ASF) under one or more

[GitHub] incubator-predictionio pull request #273: PIO-20 Merge ActionML fork

2016-08-09 Thread tom-chan
Github user tom-chan commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/273#discussion_r74125863 --- Diff: build.sbt --- @@ -163,24 +163,24 @@ pioUnidoc := { pomExtra in ThisBuild := { http