jingyimei opened a new pull request #382: DL: Naming improvements for dependent 
and independent varname
URL: https://github.com/apache/madlib/pull/382
 
 
   Previously, the name/value of 'dependent_varname' and 'indenpendent_varname'
   in madlib_keras_fit() is a bit confusing. This commit does the following:
   
   1. remove dependent_varname and independent_varname from the fit()
   interface since we know what these are from the minibatch summary table,
   and user must use minibatching for DL
   2. use original column names from the source table (i.e., pre minibatch) in
   fit summary table, not the generic names dependent_var and independent_var
   3. This change will also change the predict summary table to have the
   original column names instead of dependent_var and independent_var

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to