diff options
author | Gang Bai <me@baigang.net> | 2014-06-20 08:52:20 -0700 |
---|---|---|
committer | Xiangrui Meng <meng@databricks.com> | 2014-06-20 08:52:41 -0700 |
commit | b1ea9e5bc7f4d7000e24bdf77e9e18c29cac7193 (patch) | |
tree | 572a004da360b9113b025921a4aa63a7622d1894 /sql/hive | |
parent | 7594b3f94388a04bfe10743855ef0cb810e482d3 (diff) | |
download | spark-b1ea9e5bc7f4d7000e24bdf77e9e18c29cac7193.tar.gz spark-b1ea9e5bc7f4d7000e24bdf77e9e18c29cac7193.tar.bz2 spark-b1ea9e5bc7f4d7000e24bdf77e9e18c29cac7193.zip |
[SPARK-2163] class LBFGS optimize with Double tolerance instead of Int
https://issues.apache.org/jira/browse/SPARK-2163
This pull request includes the change for **[SPARK-2163]**:
* Changed the convergence tolerance parameter from type `Int` to type `Double`.
* Added types for vars in `class LBFGS`, making the style consistent with `class GradientDescent`.
* Added associated test to check that optimizing via `class LBFGS` produces the same results as via calling `runLBFGS` from `object LBFGS`.
This is a very minor change but it will solve the problem in my implementation of a regression model for count data, where I make use of LBFGS for parameter estimation.
Author: Gang Bai <me@baigang.net>
Closes #1104 from BaiGang/fix_int_tol and squashes the following commits:
cecf02c [Gang Bai] Changed setConvergenceTol'' to specify tolerance with a parameter of type Double. For the reason and the problem caused by an Int parameter, please check https://issues.apache.org/jira/browse/SPARK-2163. Added a test in LBFGSSuite for validating that optimizing via class LBFGS produces the same results as calling runLBFGS from object LBFGS. Keep the indentations and styles correct.
(cherry picked from commit d484ddeff1440d8e14e05c3cd7e7a18746f1a586)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
Diffstat (limited to 'sql/hive')
0 files changed, 0 insertions, 0 deletions