[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 Actually if the PR is made against `livedoc` branch the doc would be updated right away. Do you want to do that instead? Thanks. --- If your project is set up for it, you can reply

[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 and

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

2016-11-11 Thread Tom Chan
https://github.com/actionml/template-scala-parallel-universal-recommendation#v042-replaces-041 "It's meant to be used with PredictionIO-0.9.7-aml", the ActionML fork of PredictionIO. The 0.5.0 will be using the Apache Predictionio-0.10.0. I see you posted in https://github.com/PredictionIO/templa

[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 and

[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 not

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 meanwhil

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 ./incubator-predictionio-sd

Re: PIO Build Error

2016-10-12 Thread Tom Chan
[Console$] Going to run: > /home/vagrant/apache-predictionio-0.10.0-incubating/sbt/sbt > package assemblyPackageDependency > [ERROR] [Console$] [error] (*:update) sbt.ResolveException: unresolved > dependency: org.apache.predictionio#pio-build;0.9.0: not found > [ERROR] [Console$] Return code of previo

Re: PIO Build Error

2016-10-12 Thread Tom Chan
rant/apache-predictionio-0.10.0- > incubating/tapster-episode-similar2/src/main/scala/Preparator.scala:10: > not found: type PPreparator > [ERROR] [Console$] [error] extends PPreparator[TrainingData, > PreparedData] { > [ERROR] [Console$] [error] ^ > [ERROR] [Conso

Re: PIO Build Error

2016-10-12 Thread Tom Chan
There is still an instance of "io.prediction" that is not changed to "org.apache.predictionio" in your "libraryDependencies". Tom On Wed, Oct 12, 2016 at 2:06 PM, Jim Miller wrote: > Thanks for the fast response, Donald. I am still having problems tho. I > have the output and the build.sbt bel

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. We need 3 (assuming no one is -1) to > graduate > > > this. > > > > >

[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_r74126622 --- Diff: examples/scala-parallel-similarproduct/filterbyyear/engine.json --- @@ -5,6 +5,10 @@ "datasource": {

[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_r74125935 --- Diff: conf/pio-env.sh.travis --- @@ -1,22 +1,5 @@ #!/usr/bin/env bash -# -# Licensed to the Apache Software

[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_r74125884 --- Diff: conf/pio-env.sh.template --- @@ -53,7 +36,7 @@ PIO_FS_TMPDIR=$PIO_FS_BASEDIR/tmp # storage facilities. Default values are

[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_r74125874 --- Diff: conf/pio-env.sh.template --- @@ -1,22 +1,5 @@ #!/usr/bin/env bash -# -# Licensed to the Apache Software

[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

Re: The PIO JIRA instance is ready

2016-07-01 Thread Tom Chan
'tchan' Thanks, Tom On Fri, Jul 1, 2016 at 2:48 PM, Justin Yip wrote: > Hello Andrew, > > My jira account is yipjustin. > > Thanks. > > Justin > > On Fri, Jul 1, 2016 at 2:34 PM, Andrew Purtell > wrote: > > > Added! > > > > > > On Fri, Jul 1, 2016 at 2:28 PM, Kenneth Chan wrote: > > > >> Hey