[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-26 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-506073200 Thanks for the nice PR and merge now.

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-26 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-505891371 Seems all checks are pased but hang now. Maybe retrigger CI? ---

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-24 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-505269609 After offline talking, I think we can leave the performance of CLX (c5.12xlarge/24xlarge) on this page

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-23 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-504841609 > @wuxun-zhang Thank you for addressing the comments. The problem is whether we want to maintain two in

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-19 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-503806307 @anirudh2290 @ZhennanQin @ciyongch please review again if all concerns are resolved :) ---

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-15 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-502345812 @anirudh2290 would you mind reviewing this PR?

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-12 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-501512690 @szha is it ok to change the file name from inception_inference.cpp to imagenet_inference.cpp to cover

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-10 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-500694787 @wuxun-zhang could you rebase the code? ---

[GitHub] [incubator-mxnet] pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet

2019-06-06 Thread GitBox
pengzhao-intel commented on issue #15164: [C++] Improve inference script to support benchmark on Imagenet URL: https://github.com/apache/incubator-mxnet/pull/15164#issuecomment-499463852 @wuxun-zhang ping me by this ID :) Thi