[GitHub] [incubator-mxnet] stephenrawls commented on issue #14476: Change RNN OP to stateful

[GitBox <gi...@apache.org>]

stephenrawls commented on issue #14476: Change RNN OP to stateful
URL: https://github.com/apache/incubator-mxnet/pull/14476#issuecomment-475051996
 
 
   This will definitely impact my cudnn var-length PR. It's been on back burner for last couple weeks but been meaning to push it through soon.
   
   If this is going to get merged first I can just update my PR to work with this change.
   
   I'm not entirely familiar with the code base. Can you let me know what making RNN a stateful operator means? I'm not sure if this is something that I need to worry about in my PR or not.
   
   Mostly it looks like this is just taking the cudnn specific header file and putting all that code in the generic rnn-inl.h header file?

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


With regards,
Apache Git Services