-
Notifications
You must be signed in to change notification settings - Fork 29k
[SPARK-2309][MLlib] Multinomial Logistic Regression #3833
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Test build #24878 has started for PR 3833 at commit
|
|
Test build #24878 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
Test build #25072 has started for PR 3833 at commit
|
|
Test build #25072 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
Test build #25196 has started for PR 3833 at commit
|
|
Test build #25196 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
Test build #25523 has started for PR 3833 at commit
|
|
Test build #25528 has started for PR 3833 at commit
|
|
Test build #25523 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
Jenkins, please re-test again. |
|
Test FAILed. |
|
Test build #25539 has started for PR 3833 at commit
|
|
Test build #25540 has started for PR 3833 at commit
|
|
Test build #25540 has finished for PR 3833 at commit
|
|
Test PASSed. |
|
Test build #25539 has finished for PR 3833 at commit
|
|
Test FAILed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will change this API, no? is that OK?
Also optional args are a bit Java-unfriendly. Maybe at least a helper constructor here that sets nClasses == 2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
addressed. thks.
|
Test build #25565 has started for PR 3833 at commit
|
|
Test build #25565 has finished for PR 3833 at commit
|
|
Test build #26491 has finished for PR 3833 at commit
|
|
Test PASSed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to keep the original constructor to be backward compatible.
def this(weights: Vector, intercept: Double) = this(weights, intercept, weights.size, 2)
this(weights, intercept, featureSize) doesn't seem necessary to me.
|
LGTM except minor inline comments. Thanks for working on the math while taking care of precision! |
|
Test build #26531 has started for PR 3833 at commit
|
|
Test build #26531 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
Jenkins, please re-test again. |
|
Test build #26543 has started for PR 3833 at commit
|
|
Test build #26543 has finished for PR 3833 at commit
|
|
Test FAILed. |
|
test this please |
|
Test build #26549 has started for PR 3833 at commit
|
|
Test build #26549 has finished for PR 3833 at commit
|
|
Test PASSed. |
|
Merged into master. Thanks for contributing MLOR! I created https://issues.apache.org/jira/browse/SPARK-5537 for the user guide and assigned to you. |
### What changes were proposed in this pull request? Upgrade H2 from 2.1.214 to 2.2.220 [Changelog](https://www.h2database.com/html/changelog.html) ### Why are the changes needed? [CVE-2022-45868](https://nvd.nist.gov/vuln/detail/CVE-2022-45868) The following change in the release note fixes the CVE. [581ed18](h2database/h2database@581ed18) Merge pull request [#3833](https://github.com/h2database/h2database/issues/3833) from katzyn/password ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GA Closes #41963 from bjornjorgensen/h2-2.2.220. Authored-by: Bjørn Jørgensen <[email protected]> Signed-off-by: Hyukjin Kwon <[email protected]>
### What changes were proposed in this pull request? Upgrade H2 from 2.1.214 to 2.2.220 [Changelog](https://www.h2database.com/html/changelog.html) ### Why are the changes needed? [CVE-2022-45868](https://nvd.nist.gov/vuln/detail/CVE-2022-45868) The following change in the release note fixes the CVE. [581ed18](h2database/h2database@581ed18) Merge pull request [#3833](https://github.com/h2database/h2database/issues/3833) from katzyn/password ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GA Closes #41963 from bjornjorgensen/h2-2.2.220. Authored-by: Bjørn Jørgensen <[email protected]> Signed-off-by: Hyukjin Kwon <[email protected]>
#1379 is automatically closed by asfgit, and github can not reopen it once it's closed, so this will be the new PR.
Binary Logistic Regression can be extended to Multinomial Logistic Regression by running K-1 independent Binary Logistic Regression models. The following formula is implemented.
http://www.slideshare.net/dbtsai/2014-0620-mlor-36132297/25