[GitHub] [incubator-mxnet] wuxun-zhang commented on issue #16523: [MKLDNN] Fix uint quantized fc when not fusing with requantize

[GitBox <gi...@apache.org>]

wuxun-zhang commented on issue #16523: [MKLDNN] Fix uint quantized fc when not fusing with requantize
URL: https://github.com/apache/incubator-mxnet/pull/16523#issuecomment-543446828
 
 
   Sure. According to offline accuracy test for `subgraph FC`, `disable subgraph FC` and `subgraph FC w/o requantize fuse`, all accuracy numbers look good to me. 
   
   Currently, this commit can be safely merged into mkldnn-1.0 branch without conflicts (just tested locally). @TaoLv Could you help rebase mkldnn-v1.0 branch against master? Thanks
   
   
   Accuracy    | base | - | disable   subgraph FC| -| diable   fuse requantize|-
   -- | -- | --| --| -- | -- | --
   -|FP32 | INT8 | FP32 | INT8 | FP32 | INT8
   resnet18 | 70.15% | 70.02% | 70.15% | 69.90% | 70.15% | 69.90%
   resnet50 | 76.34% | 75.95% | 76.34% | 75.96% | 76.34% | 75.96%
   inceptionV3 | 77.76% | 77.82% | 77.76% | 77.83% | 77.76% | 77.83%
   suqeezenet1.0 | 56.98% | 56.80% | 56.98% | 56.80% | 56.98% | 56.80%
   mobilenet1.0 | 72.23% | 71.75% | 72.23% | 71.74% | 72.23% | 71.74%
   mobilenetV2_1.0 | 70.87% | 70.73% | 70.87% | 70.73% | 70.87% | 70.73%
   
   
   

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