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


   
![image](https://user-images.githubusercontent.com/37190647/89367810-d70ee880-d68e-11ea-8284-cfd1b30ece2e.png)
   
   According to the keras documentation, it seems there is a default value for 
optimizer as well (just that it seems to be rmsprop instead of my manually 
chosen SGD). 
   
   From my understanding of the documentation and its language, yes optimizer 
is required for compiling a keras model, but they seem to already be using a 
default optimizer (and with default params) to compile the keras model when the 
user may not specify one. 
   
   Please correct me if I missed anything or didn't understand it right.
   
   
![image](https://user-images.githubusercontent.com/37190647/89367845-edb53f80-d68e-11ea-999c-7353b33857d3.png)
   


----------------------------------------------------------------
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]


Reply via email to