You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@mxnet.apache.org by gi...@git.apache.org on 2017/10/15 01:32:37 UTC

[GitHub] szha commented on issue #8275: [WIP] Implement 'same' convolutions

szha commented on issue #8275: [WIP] Implement 'same' convolutions
URL: https://github.com/apache/incubator-mxnet/pull/8275#issuecomment-336679379
 
 
   @JulianSlzr do you prefer to limit the shape inference where asymmetric padding isn't needed in this change? In that case making the padding size inference in the frontend is likely easier. Asymmetric padding on the other hand will have to happen on each conv op.
 
----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services