You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2020/01/31 02:50:20 UTC

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #17479: Bug in RNN and LSTM with multiple layers for mxnet-mkl

pengzhao-intel commented on issue #17479: Bug in RNN and LSTM with multiple layers for mxnet-mkl
URL: https://github.com/apache/incubator-mxnet/issues/17479#issuecomment-580558742
 
 
   > @pengzhao-intel @TaoLv This problem happens on the codebase with the old MKL-DNN version (< v1.0). After [aa1074d](https://github.com/apache/incubator-mxnet/commit/aa1074dc1704d3732ab205c43d48083ef8c69680), we brought the refactor of rnn operators into MXNet, which could give the correct result. I also tested with [b5d07e3](https://github.com/apache/incubator-mxnet/commit/b5d07e30321da47d604b99048c1b57c03ec819b0) (1 commit before [aa1074d](https://github.com/apache/incubator-mxnet/commit/aa1074dc1704d3732ab205c43d48083ef8c69680)), turning out to have the wrong result. For now, we can fix it on v1.5.x branch only. Shall we?
   
   Since the fix is in v1.6.x, I suggest switching to v1.6.x. Actually, it depends on their deployment requirements. If it is necessary, we can evaluate how difficult to backport to v1.5.x.

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