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

Posted to commits@mxnet.apache.org by GitBox <gi...@apache.org> on 2019/01/23 07:51:47 UTC

[GitHub] suyz526 commented on issue #13955: adding the gluon implementation of deformable convolution

suyz526 commented on issue #13955: adding the gluon implementation of deformable convolution
URL: https://github.com/apache/incubator-mxnet/pull/13955#issuecomment-456703521
 
 
   Hi @marcoabreu!
   
   The ndarray and symbol implementation of the deformable convolution already exist. Here they are wrapped with gluon hybridblock. 
   
   I have replaced some nn.Conv2D layer in a network with my gluon implementation of deformable convolution. And the loss also reduced. So I think it works.
   
   Maybe I understood the item "All changes have test coverage" wrong. Should I do some further test?
   
   

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