[GitHub] madlib pull request #272: MLP: Add momentum and nesterov to gradient updates...

[kaknikhil]

Github user kaknikhil commented on a diff in the pull request:

    https://github.com/apache/madlib/pull/272#discussion_r191945714
  
    --- Diff: src/modules/convex/task/mlp.hpp ---
    @@ -197,6 +244,7 @@ MLP<Model, Tuple>::loss(
             const model_type                    &model,
             const independent_variables_type    &x,
             const dependent_variable_type       &y_true) {
    +
    --- End diff --
    
    +1 will do


---