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/02/27 21:33:54 UTC

[GitHub] stu1130 edited a comment on issue #14259: Add Gluon Transformer Crop

stu1130 edited a comment on issue #14259: Add Gluon Transformer Crop
URL: https://github.com/apache/incubator-mxnet/pull/14259#issuecomment-468039343
 
 
   @anirudhacharya yes it is
   For the first one, the resize functionality is moved out of Crop operator so no .value() is needed. And I added the usage and doc for it.
   For the second one, if we want to make fixed_crop take advantage of my operator ,it would be one line change from using the nd.crop to nd.image.crop. Nothing sinificant change. so I kept the original one.

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