fmcquillan99 edited a comment on pull request #506:
URL: https://github.com/apache/madlib/pull/506#issuecomment-668952722


   for (9) 
   
   Keras docs say that optimizer and loss function are mandatory params, so I 
don't think we should put SGD if user does not specify an optimizer.  We should 
let it fail, no?
   
   https://keras.io/api/optimizers/
   `An optimizer is one of the two arguments required for compiling a Keras 
model`


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to