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

2020-07-27 Thread GitBox
kpuatamazon commented on issue #18699: URL: https://github.com/apache/incubator-mxnet/issues/18699#issuecomment-664253978 Ran these in a loop over the weekend with 136 samples. keep 83b5170 | a | b | c | justc++ |   -- | -- | -- | -- | -- | -- 301.324 | 298.468 | 294.094 |

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

2020-07-24 Thread GitBox
kpuatamazon commented on issue #18699: URL: https://github.com/apache/incubator-mxnet/issues/18699#issuecomment-663748364 3 runs each sorted by increasing time taken. All downgrade MKLDNN to cb2cc7a + force MKL backend hack. | time | time | time | git checkout 8e39518 ... |

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

2020-07-24 Thread GitBox
kpuatamazon commented 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: