dcslin commented on pull request #697: URL: https://github.com/apache/singa/pull/697#issuecomment-633231970
One of the caveat worth noting for user is that args of `initialize()`, `forward()` need to match exactly to `__call__()` Normally `__call__(x)` is the same as `initialize(x)`, which is ok. But for `__call__(x, h0, c0)`, `initialize()` need to tweak a bit to be: `def initialize(self, h0, c0):` though h0, c0 is not required. fixing some code in LSTM class and testing ---------------------------------------------------------------- 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
