GitHub user feynmanliang opened a pull request:
https://github.com/apache/spark/pull/10743
[SPARK-12804][ML] Fix LogisticRegression with FitIntercept on all same
label training data
You can merge this pull request into a Git repository by running:
$ git pull https://github.co
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-171409858
**[Test build #49325 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49325/consoleFull)**
for PR 10743 at commit
[`caf7a1b`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-171421277
**[Test build #49325 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49325/consoleFull)**
for PR 10743 at commit
[`caf7a1b`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-171421490
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-171421491
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49649843
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -339,9 +339,11 @@ class LogisticRegression @Since("1.2.0")
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49650452
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,22 @@ class LogisticRegressionSuite
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49652270
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,22 @@ class LogisticRegressionSuite
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49946338
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,22 @@ class LogisticRegressionSuite
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49946694
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -339,9 +339,11 @@ class LogisticRegression @Since("1.
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172328753
@dbtsai @jkbradley ready for second review.
The big diff is because I grouped the same label cases with the normal case
to generate `coefficients`, `interc
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172329504
**[Test build #49554 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49554/consoleFull)**
for PR 10743 at commit
[`d676f62`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172333931
**[Test build #49554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49554/consoleFull)**
for PR 10743 at commit
[`d676f62`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172334065
Merged build finished. Test PASSed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172334066
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49955604
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,27 @@ class LogisticRegressionSuite
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49955619
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,27 @@ class LogisticRegressionSuite
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49955755
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since("1.2.0
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172401612
LGTM except couple minor issues. 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
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49955925
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since("1.2.0
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r49955940
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,27 @@ class LogisticRegressionSuite
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50006902
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since(
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50006858
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since(
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50007441
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/classification/LogisticRegressionSuite.scala
---
@@ -883,6 +884,27 @@ class LogisticRegressionSuite
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172556919
@dbtsai Added `fitIntercept=false` tests and fixed comments/`logWarning`
messages.
---
If your project is set up for it, you can reply to this email and have your
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172561357
**[Test build #49597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49597/consoleFull)**
for PR 10743 at commit
[`0f4824d`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172561751
Merged build finished. Test FAILed.
---
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172561755
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172561744
**[Test build #49597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49597/consoleFull)**
for PR 10743 at commit
[`0f4824d`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172594732
**[Test build #49600 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49600/consoleFull)**
for PR 10743 at commit
[`c8c1586`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172606348
**[Test build #49600 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49600/consoleFull)**
for PR 10743 at commit
[`c8c1586`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172606483
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172606481
Merged build finished. Test PASSed.
---
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 projec
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50032693
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since("1.2.0
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50032721
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since("1.2.0
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50041068
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since(
Github user feynmanliang commented on a diff in the pull request:
https://github.com/apache/spark/pull/10743#discussion_r50041066
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/classification/LogisticRegression.scala
---
@@ -276,113 +276,123 @@ class LogisticRegression @Since(
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172802711
**[Test build #49684 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49684/consoleFull)**
for PR 10743 at commit
[`95816d4`](https://gi
Github user feynmanliang commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172811882
@dbtsai validating coefficients with R will be harder than expected,
`glmnet` requires feature dimension >= 2 and `glm` doesn't yield +/- Infinity
intercepts when
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172815986
**[Test build #49684 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/49684/consoleFull)**
for PR 10743 at commit
[`95816d4`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172816195
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172816192
Merged build finished. Test PASSed.
---
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 projec
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172954097
Ideally, we would like to see the one without intercept to ensure that
nothing will not be messed up during the refactoring. I'll merge now, and let's
add the test for a
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172954547
Merged into master. 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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10743
---
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 ena
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/10743#issuecomment-172965625
A followup PR is created: https://issues.apache.org/jira/browse/SPARK-12908
---
If your project is set up for it, you can reply to this email and have your
reply appear
46 matches
Mail list logo