vpirogov commented on a change in pull request #20606:
URL: https://github.com/apache/incubator-mxnet/pull/20606#discussion_r721515281
##########
File path: python/mxnet/contrib/quantization.py
##########
@@ -552,9 +552,9 @@ def quantize_model_mkldnn(sym, arg_params, aux_params,
data_names=('data',),
raise ValueError('currently only supports single ctx, while received
%s' % str(ctx))
if ctx.device_type != 'cpu':
raise ValueError(
- 'quantize_model_mkldnn only support Intel cpu platform with
MKL-DNN Backend')
+ 'quantize_model_dnnl only support Intel cpu platform with DNNL
Backend')
Review comment:
I would suggest to use `quantize_model_onednn` instead of
`quantie_model_dnnl`. There's no need to introduce naming with `dnnl` unless it
was already there.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
