[GitHub] madlib pull request #240: MLP: Fix step size initialization based on learnin...

[kaknikhil]

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

    https://github.com/apache/madlib/pull/240#discussion_r175224785
  
    --- Diff: src/ports/postgres/modules/convex/mlp_igd.py_in ---
    @@ -112,6 +112,7 @@ def mlp(schema_madlib, source_table, output_table, 
independent_varname,
         num_output_nodes = 0
         classes = []
         dependent_type = get_expr_type(dependent_varname, source_table)
    +    classes_str = None
    --- End diff --
    
    we don't really need this `classes_str` variable, right ?


---