piiswrong commented on a change in pull request #7352: add Sequential 
compatibility to rnn layers
URL: https://github.com/apache/incubator-mxnet/pull/7352#discussion_r132317845
 
 

 ##########
 File path: python/mxnet/gluon/rnn/rnn_layer.py
 ##########
 @@ -286,6 +293,8 @@ class RNN(_RNNLayer):
         The recurrent state's shape is `(num_layers, batch_size, num_hidden)`.
         If `bidirectional` is True, state shape will instead be
         `(2*num_layers, batch_size, num_hidden)`
+        If input recurrent state is None, zeros are used as default begin 
states,
 
 Review comment:
   Recurrent state shape -> Recurrent state.
   
   and change the beginning sentence to The recurrent states is an NDArray with 
shape xxx.
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

Reply via email to