trevor-m opened a new pull request #7885:
URL: https://github.com/apache/tvm/pull/7885


   Previously TVM would throw an error if an operator was using SAME padding 
and also had a dynamic H or W dimension:
   `unsupported operand type(s) for %: 'Any' and 'int'`
   
   However, if stride == 1, then the input dimension value is actually not 
needed to compute the padding values.
   Also added explicit message for the remaining unsupported case (strides != 
1).
   Added a test case for pooling, but this change helps all of the following 
operators:
   Conv2D
   Conv2DBackpropInput
   DepthwiseConv2dNative
   Conv3D
   Conv3DBackpropInputV2
   AvgPool
   MaxPool
   AvgPool3D
   MaxPool3D
   Dilation2D
   
   
   If strides is not 1, we currently don't have a way to support the dynamic 
padding. I did some experiments, but it seems type inferencing through 
`dyn.nn.pad` is not sufficient because it doesn't know that only H and W 
dimensions are padded.


-- 
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]


Reply via email to