szha commented on issue #18942: URL: https://github.com/apache/incubator-mxnet/issues/18942#issuecomment-678905999
@ekdnam yes. I think in this case the problem is that both stride1 and stride2 should not be zero. The existing mechanism for declaring operator parameter ranges is in the declaration of the parameter object. Currently, it only has default values: https://github.com/apache/incubator-mxnet/blob/0de7484884292eb028342b1e5669233792429af0/src/operator/correlation-inl.h#L57-L60 To fix it, we will need to do something similar to https://github.com/apache/incubator-mxnet/pull/18857/files ---------------------------------------------------------------- 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]
