[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-28 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8399 --- 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, or if the feature is enab

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135656766 No, this LGTM. I'll merge this with branch-1.5 and master now. Thanks very much! --- If your project is set up for it, you can reply to this email and have your rep

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread noel-smith
Github user noel-smith commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135655500 That would be great - I've just messaged him. If there are any other changes you need to get this into 1.5 I'll get them in ASAP today. --- If your project is set up

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135605403 [Test build #1700 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1700/console) for PR 8399 at commit [`bada453`](https://github

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135603184 [Test build #1700 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1700/consoleFull) for PR 8399 at commit [`bada453`](https://git

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135603112 Ping @mengxr In case I can't check this soon, it would be great to get this into 1.5 if there is an RC3. --- If your project is set up for it, you can reply to this e

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135576579 [Test build #1699 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1699/console) for PR 8399 at commit [`7794cf7`](https://github

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-27 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135576535 [Test build #1699 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1699/consoleFull) for PR 8399 at commit [`7794cf7`](https://git

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135204427 [Test build #1695 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1695/console) for PR 8399 at commit [`7794cf7`](https://github

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135204382 [Test build #1695 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/1695/consoleFull) for PR 8399 at commit [`7794cf7`](https://git

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135204119 Looks good except for those 2 items. --- 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 do

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/8399#discussion_r38047496 --- Diff: python/pyspark/ml/evaluation.py --- @@ -85,6 +88,9 @@ def _evaluate(self, dataset): self._transfer_params_to_java() retur

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread jkbradley
Github user jkbradley commented on a diff in the pull request: https://github.com/apache/spark/pull/8399#discussion_r38047455 --- Diff: python/pyspark/ml/evaluation.py --- @@ -66,6 +66,9 @@ def evaluate(self, dataset, params=None): else: raise ValueErro

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135195804 Thanks for the PR! I'll take a look now. --- 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 proje

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-26 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-135195777 ok to test --- 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 e

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-24 Thread noel-smith
GitHub user noel-smith opened a pull request: https://github.com/apache/spark/pull/8399 [SPARK-10188] [Pyspark] Pyspark CrossValidator with RMSE selects incorrect model * Added isLargerBetter() method to Pyspark Evaluator to match the Scala version. * JavaEvaluator delegates is

[GitHub] spark pull request: [SPARK-10188] [Pyspark] Pyspark CrossValidator...

2015-08-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8399#issuecomment-134344428 Can one of the admins verify this patch? --- 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 pr