[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-03-01 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76623310 [Test build #28151 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28151/consoleFull) for PR 4811 at commit [`f135dac`](https://githu

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76489837 Made JIRA: [https://issues.apache.org/jira/browse/SPARK-6071] --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76487495 LGTM. I ran into a bug running the example, but it seems to be coming from elsewhere. It happens when calling train---and not all the time, only sometimes: `

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76485638 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76485628 [Test build #28088 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28088/consoleFull) for PR 4811 at commit [`06140a4`](https://gith

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4811#discussion_r25544222 --- Diff: python/pyspark/mllib/util.py --- @@ -168,6 +168,64 @@ def loadLabeledPoints(sc, path, minPartitions=None): return callMLlibFunc("loadL

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/4811#discussion_r25544216 --- Diff: docs/mllib-collaborative-filtering.md --- @@ -220,6 +218,10 @@ predictions = model.predictAll(testdata).map(lambda r: ((r[0], r[1]), r[2])) r

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76471927 [Test build #28088 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28088/consoleFull) for PR 4811 at commit [`06140a4`](https://githu

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76471611 If we have couple days before RC2, this would be nice to have. We use the same API as in Scala/Java and there is no real implementation in this PR. Having save/load would

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76446225 I messed up not passing sc to save/load. Is this patch going into 1.3? If not, then I'll submit a separate patch fixing the documentation (which will conflict a littl

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76360565 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76360555 [Test build #28056 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28056/consoleFull) for PR 4811 at commit [`282ec8d`](https://gith

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/4811#issuecomment-76352594 [Test build #28056 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28056/consoleFull) for PR 4811 at commit [`282ec8d`](https://githu

[GitHub] spark pull request: [SPARK-5991][MLLIB] support save/load in PySpa...

2015-02-26 Thread mengxr
GitHub user mengxr opened a pull request: https://github.com/apache/spark/pull/4811 [SPARK-5991][MLLIB] support save/load in PySpark's ALS A simple wrapper in Python to save/load `MatrixFactorizationModel` in Python. @jkbradley You can merge this pull request into a Git repository