crazy-cat commented on a change in pull request #7393: add depthwise convolution's gpu version optimization URL: https://github.com/apache/incubator-mxnet/pull/7393#discussion_r132097843
########## File path: src/operator/convolution-inl.h ########## @@ -97,6 +98,8 @@ struct ConvolutionParam : public dmlc::Parameter<ConvolutionParam> { .set_default(dmlc::optional<int>()) .describe("Set layout for input, output and weight. Empty for\n " "default layout: NCW for 1d, NCHW for 2d and NCDHW for 3d."); + DMLC_DECLARE_FIELD(depthwise_conv_off).set_default(false) + .describe("whether to turn off depthwise convolution for this layer"); Review comment: just provide a choice, like the cudnn_off ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services