joddiy opened a new issue #583: The padding of Conv2d and Pool22 not support SAME_UPPER, SAME_LOWER URL: https://github.com/apache/singa/issues/583 Now the singa's padding is, we assign a value, and it adds the amount of value's padding at both sides of head and tail. However, according to the [onnx doc](https://github.com/onnx/onnx/blob/master/docs/Operators.md#Conv), there are four types of padding method, which are NOTSET, SAME_UPPER, SAME_LOWER or VALID. Singa cannot support SAME_UPPER, SAME_LOWER yet. > SAME_UPPER or SAME_LOWER mean pad the input so that the output spatial size match the input. In case of an odd number add the extra padding at the end for SAME_UPPER and at the beginning for SAME_LOWER. For example, the input is 32*32, the stride is 1, the kernel is 4*4, and output is same, we need to add (3,3) zeros totally, each side is 3/2 = 1.5, cannot work base on the current setting of singa.
---------------------------------------------------------------- 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
