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/07/24 14:22:12 UTC

[GitHub] [incubator-mxnet] kpuatamazon edited a comment on issue #18699: Simplified HybridBlock.forward commit made Sockeye 4% slower

kpuatamazon edited a comment on issue #18699:
URL: https://github.com/apache/incubator-mxnet/issues/18699#issuecomment-663563376


   I split commit 83b5170 into just C++ changes and pushed to this branch so you can see precisely what it looks like: https://github.com/kpuatamazon/incubator-mxnet/tree/split-hybrid .  The C++ changes made it faster then the Python changes made it slower.  
   
   Similar experimental setup.  
   
   | Time (s) | Commit |
   |--|--|
   | 288.385 | 83b5170 + downgrade MKLDNN to cb2cc7a  |
   | 271.170 | 8e39518 + downgrade MKLDNN to cb2cc7a + only the C++ changes from 83b5170 at https://github.com/kpuatamazon/incubator-mxnet/tree/split-hybrid |
   | 281.465 | 8e39518 + downgrade MKLDNN to cb2cc7a |
   


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