diff options
author | DB Tsai <dbtsai@alpinenow.com> | 2014-03-02 00:31:59 -0800 |
---|---|---|
committer | Reynold Xin <rxin@apache.org> | 2014-03-02 00:31:59 -0800 |
commit | 6fc76e49c19310ec0d6cdf4754271ad09d652576 (patch) | |
tree | 89982b69fda4c22f2be37f516597dc3edfa3ac19 /sbt | |
parent | 3a8b698e961ac05d9d53e2bbf0c2844fcb1010d1 (diff) | |
download | spark-6fc76e49c19310ec0d6cdf4754271ad09d652576.tar.gz spark-6fc76e49c19310ec0d6cdf4754271ad09d652576.tar.bz2 spark-6fc76e49c19310ec0d6cdf4754271ad09d652576.zip |
Initialized the regVal for first iteration in SGD optimizer
Ported from https://github.com/apache/incubator-spark/pull/633
In runMiniBatchSGD, the regVal (for 1st iter) should be initialized
as sum of sqrt of weights if it's L2 update; for L1 update, the same logic is followed.
It maybe not be important here for SGD since the updater doesn't take the loss
as parameter to find the new weights. But it will give us the correct history of loss.
However, for LBFGS optimizer we implemented, the correct loss with regVal is crucial to
find the new weights.
Author: DB Tsai <dbtsai@alpinenow.com>
Closes #40 from dbtsai/dbtsai-smallRegValFix and squashes the following commits:
77d47da [DB Tsai] In runMiniBatchSGD, the regVal (for 1st iter) should be initialized as sum of sqrt of weights if it's L2 update; for L1 update, the same logic is followed.
Diffstat (limited to 'sbt')
0 files changed, 0 insertions, 0 deletions