[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193410022 ## File path: tests/python/quantization/test_quantization.py ## @@ -120,114 +128,126

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193411184 ## File path: tests/python/quantization/test_quantization.py ## @@ -409,11 +441,19

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193410215 ## File path: tests/python/quantization/test_quantization.py ## @@ -319,6 +339,11 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193410323 ## File path: tests/python/quantization/test_quantization.py ## @@ -409,11 +441,19

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193410255 ## File path: tests/python/quantization/test_quantization.py ## @@ -353,6 +378,11 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193409350 ## File path: tests/python/mkl/test_quantization_mkldnn.py ## @@ -0,0 +1,28 @@ +#

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-06-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r193410162 ## File path: tests/python/quantization/test_quantization.py ## @@ -120,114 +128,126

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-30 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r191751428 ## File path: ci/docker/runtime_functions.sh ## @@ -382,7 +381,6 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-30 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r191752075 ## File path: ci/docker/runtime_functions.sh ## @@ -381,13 +381,12 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190770998 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,12 +199,11

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190770962 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -159,7 +159,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190491572 ## File path: src/operator/quantization/dequantize.cc ## @@ -50,7 +70,12 @@ by keep

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190491349 ## File path: src/operator/quantization/quantize.cc ## @@ -61,7 +81,12 @@ where

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190488717 ## File path: src/operator/quantization/quantized_pooling.cc ## @@ -78,19 +81,39 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190488062 ## File path: src/operator/quantization/quantized_flatten-inl.h ## @@ -62,11 +62,21

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190488482 ## File path: src/operator/quantization/quantized_flatten-inl.h ## @@ -96,10 +106,10

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190484093 ## File path: src/operator/quantization/quantized_conv.cc ## @@ -86,12 +89,10 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-23 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190447983 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -159,7 +159,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-23 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r190448136 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,12 +199,11

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-20 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r189500075 ## File path: src/operator/nn/mkldnn/mkldnn_convolution.cc ## @@ -162,73 +165,51 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-20 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r18948 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,12 +199,11

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-20 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r189499693 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -159,7 +159,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-16 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r188569030 ## File path: include/mxnet/c_api.h ## @@ -1423,13 +1423,15 @@ MXNET_DLL int

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-16 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r188534806 ## File path: src/operator/nn/mkldnn/mkldnn_convolution-inl.h ## @@ -0,0 +1,100 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-05-16 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r188510242 ## File path: src/operator/quantization/requantize-inl.h ## @@ -68,6 +68,24 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-27 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r184622443 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_conv.cc ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-27 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r184622611 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-27 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r184621143 ## File path: src/operator/quantization/mkldnn/mkldnn_dequantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-27 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r184622800 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -129,7 +129,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241106 ## File path: src/operator/quantization/mkldnn/mkldnn_quantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241662 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -129,7 +129,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183759204 ## File path: src/operator/quantization/dequantize.cc ## @@ -23,11 +23,32 @@ *

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183758965 ## File path: include/mxnet/c_api.h ## @@ -1423,13 +1423,15 @@ MXNET_DLL int

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-24 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183753288 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,7 +198,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241376 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_conv.cc ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241805 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -198,7 +198,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241122 ## File path: src/operator/quantization/mkldnn/mkldnn_quantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241596 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241614 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241662 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -129,7 +129,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241827 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -250,6 +250,165

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241566 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241525 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183240995 ## File path: src/operator/quantization/mkldnn/mkldnn_dequantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241106 ## File path: src/operator/quantization/mkldnn/mkldnn_quantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183240951 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -211,6 +211,7 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241649 ## File path: src/operator/quantization/mkldnn/mkldnn_requantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241432 ## File path: src/operator/quantization/mkldnn/mkldnn_quantized_pooling.cc ## @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241679 ## File path: src/operator/quantization/quantize_graph_pass.cc ## @@ -159,7 +159,7

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183241068 ## File path: src/operator/quantization/mkldnn/mkldnn_dequantize-inl.h ## @@ -0,0

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183240942 ## File path: src/c_api/c_api_symbolic.cc ## @@ -595,7 +596,12 @@ int

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183240784 ## File path: include/mxnet/c_api.h ## @@ -1409,13 +1409,17 @@ MXNET_DLL int

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-22 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r183240802 ## File path: python/mxnet/contrib/quantization.py ## @@ -72,7 +72,7 @@ def

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r179790587 ## File path: src/operator/nn/mkldnn/mkldnn_base.cc ## @@ -211,6 +211,7 @@

[GitHub] jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization

2018-04-06 Thread GitBox
jinhuang415 commented on a change in pull request #10433: [MXNET-290] MKLDNN support for model quantization URL: https://github.com/apache/incubator-mxnet/pull/10433#discussion_r179790862 ## File path: src/operator/nn/pooling.cc ## @@ -368,7 +368,11 @@ height, width)*.