Github user mengxr commented on the pull request:
https://github.com/apache/incubator-spark/pull/633#issuecomment-35915806
@dbtsai Since regVal remains 0.0 for any existing updater in MLlib, it
would make more sense if this change comes with the L-BFGS PR you are working
on.
---
If
GitHub user dbtsai opened a pull request:
https://github.com/apache/incubator-spark/pull/633
Initialized the regVal for first iteration in SGD optimizer
In runMiniBatchSGD, the regVal (for 1st iter) should be initialized
as sum of sqrt of weights if it's L2 update; for L1 update,
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/incubator-spark/pull/633#issuecomment-35800239
Can one of the admins verify this patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. To