anko-intel commented on a change in pull request #20137:
URL: https://github.com/apache/incubator-mxnet/pull/20137#discussion_r609507854
##########
File path: tests/python/mkl/test_mkldnn.py
##########
@@ -482,6 +481,10 @@ def check_Deconvolution_training(stype):
elif np.array(shape).shape[0] == 4:
test = mx.symbol.Deconvolution(data=data, kernel=(3, 3),
stride=(2, 2), num_filter=4)
weight_tmp = np.random.normal(-0.1, 0.1, size=(3, 4, 3, 3))
+ # not testing stype fallback, as 3D deconvolution is not natively
supported yet
Review comment:
You are right, so maybe you can change comment to
```suggestion
# not testing row_sparse stype, as 3D deconvolution is not
natively supported
```
--
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:
[email protected]
