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
[SPARK-3078][MLLIB] Make LRWithLBFGS API consistent with others #1973
Conversation
QA tests have started for PR 1973 at commit
|
QA tests have finished for PR 1973 at commit
|
Jenkins, retest this please. |
QA tests have started for PR 1973 at commit
|
def setMaxNumIterations(iters: Int): this.type = { | ||
this.setNumCorrections(iters) |
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.
Should it be
this. setNumIterations(iters)
QA tests have finished for PR 1973 at commit
|
QA tests have started for PR 1973 at commit
|
QA tests have started for PR 1973 at commit
|
QA tests have finished for PR 1973 at commit
|
QA tests have finished for PR 1973 at commit
|
LGTM. Merged into both master and branch-1.1. Thanks!! |
Should ask users to set parameters through the optimizer. dbtsai Author: Xiangrui Meng <meng@databricks.com> Closes #1973 from mengxr/lr-lbfgs and squashes the following commits: e3efbb1 [Xiangrui Meng] fix tests 21b3579 [Xiangrui Meng] fix method name 641eea4 [Xiangrui Meng] Merge remote-tracking branch 'apache/master' into lr-lbfgs 456ab7c [Xiangrui Meng] update LRWithLBFGS (cherry picked from commit 5d25c0b) Signed-off-by: Xiangrui Meng <meng@databricks.com>
Should ask users to set parameters through the optimizer. dbtsai Author: Xiangrui Meng <meng@databricks.com> Closes apache#1973 from mengxr/lr-lbfgs and squashes the following commits: e3efbb1 [Xiangrui Meng] fix tests 21b3579 [Xiangrui Meng] fix method name 641eea4 [Xiangrui Meng] Merge remote-tracking branch 'apache/master' into lr-lbfgs 456ab7c [Xiangrui Meng] update LRWithLBFGS
Should ask users to set parameters through the optimizer. @dbtsai