jmerkow commented on issue #9730: Check padding size for global pooling URL: https://github.com/apache/incubator-mxnet/pull/9730#issuecomment-494224507 The major issue is that it needlessly breaks backward compatibility. Why force it to zero? Perhaps a warning if padding is set to non-zero? Or a separate GlobalPooling layer that doesn't have that parameter. I don't understand the logic with forcing it to zero. What exactly does it break if its a non-zero value?
---------------------------------------------------------------- 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] With regards, Apache Git Services
