[GitHub] KellenSunderland edited a comment on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-01-15 Thread GitBox
KellenSunderland edited a comment on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-454660795 Yes that's what I'd suggest as well. I would open a PR with the API changes you like to make and enough

[GitHub] KellenSunderland edited a comment on issue #13697: [MKLDNN] Enable signed int8 support for convolution.

2019-01-01 Thread GitBox
KellenSunderland edited a comment on issue #13697: [MKLDNN] Enable signed int8 support for convolution. URL: https://github.com/apache/incubator-mxnet/pull/13697#issuecomment-450708291 @ZhennanQin Thanks to you for the great contribution! I agree your change will fix the compile, but I'm