[GitHub] incubator-predictionio issue #24: Readme: Quickstart Link leads to 404

2017-02-09 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/issues/24 My apologies, I temporarily broke the site. Should be good now. On Thu, Feb 9, 2017 at 8:02 AM, Vinayaktech <notificati...@github.com> wrote: > t

[GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2017-01-17 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/328 @pferrel: Yes the hash of engine directory path is used as engineVersion (which kind of acts as the id). However, I'm working on making updates to this PR so that users

[GitHub] incubator-predictionio pull request #279: [PIO-26] Integrate Apache RAT for ...

2016-08-19 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/279 [PIO-26] Integrate Apache RAT for license checking You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator

[GitHub] incubator-predictionio issue #279: [PIO-26] Integrate Apache RAT for license...

2016-08-24 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/279 I'm not sure if license check is the reason for the delay. As you can see here (https://travis-ci.org/apache/incubator-predictionio/builds/153982829) there's is little difference

[GitHub] incubator-predictionio issue #293: io.prediction#core_2.10;0.10.0-SNAPSHOT: ...

2016-09-09 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/issues/293 Hi, please redirect usage questions to u...@predictionio.incubator.apache.org for more exposure to the rest of the community. Also, could you tell me which engine template you

[GitHub] incubator-predictionio pull request #294: [PIO-26] BUG: Put license before X...

2016-09-15 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/294 [PIO-26] BUG: Put license before XML declaration This creates errors when updating Docker image You can merge this pull request into a Git repository by running: $ git pull

[GitHub] incubator-predictionio-template-text-classifier pull request #6: Update name...

2016-10-07 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio-template-text-classifier/pull/6 Update namespace for Apache PredictionIO v0.10 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-predictionio-template-skeleton pull request #4: Update namespace f...

2016-10-07 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio-template-skeleton/pull/4 Update namespace for Apache PredictionIO v0.10 You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514

[GitHub] incubator-predictionio-template-recommender pull request #10: Update namespa...

2016-10-07 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio-template-recommender/pull/10 Update namespace for Apache PredictionIO v0.10 You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-predictionio pull request #300: [PIO-35] Add integration tests for...

2016-09-21 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/300 [PIO-35] Add integration tests for official templates **Changes:** - Integration test fetches templates from Github, instead of storing local copy. - Install git

[GitHub] incubator-predictionio pull request #323: [PIO-40] Remove docs/manual/obsole...

2016-11-04 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/323 [PIO-40] Remove docs/manual/obsolete/* Remove obsolete docs, along with small changes in .gitignore for ignoring integration test artifacts You can merge this pull request

[GitHub] incubator-predictionio pull request #325: Add git to Dockerfile

2016-11-07 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/325 Add git to Dockerfile For fetching templates via github You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator

[GitHub] incubator-predictionio issue #304: PIO-39: All *.md files are missing Apache...

2016-10-11 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/304 @smarthi Right okay. It seems like they'll be purged with a seperate PR, and maybe we can remove "*.md" from .rat-excludes following the obsolete/ purge --- If yo

[GitHub] incubator-predictionio issue #304: PIO-39: All *.md files are missing Apache...

2016-10-11 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/304 I've run RAT license checking after removing .md files from .rat_excludes, and it seems that some files are missing license headers. The results can be seen here: https://travis

[GitHub] incubator-predictionio-template-text-classifier pull request #7: Update temp...

2016-10-13 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio-template-text-classifier/pull/7 Update template for PIO version 0.10.0-incubating You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] incubator-predictionio-template-text-classifier pull request #6: Update name...

2016-10-13 Thread chanlee514
Github user chanlee514 closed the pull request at: https://github.com/apache/incubator-predictionio-template-text-classifier/pull/6 --- 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

[GitHub] incubator-predictionio issue #307: [PIO-39] Add license header for .erb file...

2016-10-12 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/307 Once this is merged, I'll create a new PR for removing obsolete docs --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] incubator-predictionio pull request #300: [PIO-35] Add integration tests for...

2016-10-12 Thread chanlee514
Github user chanlee514 closed the pull request at: https://github.com/apache/incubator-predictionio/pull/300 --- 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 wishes so

[GitHub] incubator-predictionio pull request #307: [PIO-39] Add license header for .e...

2016-10-12 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/307 [PIO-39] Add license header for .erb files You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator-predictionio

[GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2016-12-01 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/328 Right now, the commands throw java.io.FileNotFoundException regarding engine.json. May be better if we throw an error message saying the commands should be executed inside engine

[GitHub] incubator-predictionio issue #328: [PIO-47] Eliminate enginemanifest for sta...

2016-12-01 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/328 @pferrel: I agree, and that would be possible with a general metadata registry server which would be the next step. The reason I started by removing manifest.json is because

[GitHub] incubator-predictionio issue #345: [PIO-30] Set up a cross build for Spark 2...

2017-03-03 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/345 Hi @shimamoto, thanks for pointing that out. I'll make changes to enable `-Dbuild.profile=scala-2.11 -Dspark.version=1.6.0`, which is a simple matter of handling the build args

[GitHub] incubator-predictionio pull request #357: [PIO-51] Properly locate engine di...

2017-03-08 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/357 [PIO-51] Properly locate engine directory When --engine-dir is passed as CLI argument, use this instead of user.dir You can merge this pull request into a Git repository

[GitHub] incubator-predictionio pull request #357: [PIO-51] Properly locate engine di...

2017-03-08 Thread chanlee514
Github user chanlee514 commented on a diff in the pull request: https://github.com/apache/incubator-predictionio/pull/357#discussion_r105086596 --- Diff: tools/src/main/scala/org/apache/predictionio/tools/console/Console.scala --- @@ -734,7 +734,7 @@ object Console extends

[GitHub] incubator-predictionio issue #401: [PIO-72] Fix class loading for pio-shell

2017-07-12 Thread chanlee514
Github user chanlee514 commented on the issue: https://github.com/apache/incubator-predictionio/pull/401 @mars Certainly. I've already merged it for my branch [here](https://github.com/chanlee514/incubator-predictionio/commits/pr-372). I still haven't had time to look into the shade

[GitHub] incubator-predictionio pull request #385: [PIO-72] Fix class loading for pio...

2017-06-28 Thread chanlee514
Github user chanlee514 closed the pull request at: https://github.com/apache/incubator-predictionio/pull/385 --- 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 wishes so

[GitHub] incubator-predictionio pull request #401: [PIO-72] Fix class loading for pio...

2017-06-28 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/401 [PIO-72] Fix class loading for pio-shell You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator-predictionio

[GitHub] incubator-predictionio pull request #385: [PIO-72] Fix class loading for pio...

2017-05-26 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/385 [PIO-72] Fix class loading for pio-shell You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator-predictionio

[GitHub] incubator-predictionio pull request #385: [PIO-72] Fix class loading for pio...

2017-06-02 Thread chanlee514
GitHub user chanlee514 reopened a pull request: https://github.com/apache/incubator-predictionio/pull/385 [PIO-72] Fix class loading for pio-shell You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator-predictionio

[GitHub] incubator-predictionio pull request #439: [PIO-131] Fix Apache licensing iss...

2017-10-05 Thread chanlee514
GitHub user chanlee514 opened a pull request: https://github.com/apache/incubator-predictionio/pull/439 [PIO-131] Fix Apache licensing issues for doc site You can merge this pull request into a Git repository by running: $ git pull https://github.com/chanlee514/incubator