dcslin commented on pull request #697:
URL: https://github.com/apache/singa/pull/697#issuecomment-633374596


   > there is no params explicitly defined by MyModel; therefore there should 
be no direct params of MyModel. All params are from the sublayers of MyModel, 
i.e., l1 and l2.
   
   There is a case for params explicitly defined by MyModel, for example, singa 
native RNN implementation by default let the model handle the initial 
states/hidden value like:
   https://github.com/apache/singa/blob/master/examples/rnn/train.py#L47-L48
   


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