You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2019/11/24 23:43:11 UTC

[GitHub] [incubator-tvm] yongwww opened a new pull request #4417: [TOPI][RELAY][OP] add op crop_and_resize

yongwww opened a new pull request #4417: [TOPI][RELAY][OP] add op crop_and_resize
URL: https://github.com/apache/incubator-tvm/pull/4417
 
 
   Add `crop_and_resize` op into topi and relay and TensorFlow frontend. 
   
   Some of the reasons why we need `crop_and_resize` are as below:
   
   - `CropAndResize` is being used in TensorFlow frequently in both training and inference, 'almost' all models (such as ssd, maskrcnn, etc.) in tensorflow object detection zoo are using this op.
   - It is possible to use `strided_slice`, `resize` and `concatenate` to compose `crop_and_resize` for some cases,  but the composition is not able to cover all cases, for example, when  `extrapolation_value` is used, `boxes` and/or `box_indices` are not list, etc. 
   - Most importantly, using `strided_slice`, `resize` and `concatenate` to build `CropAndResize` will introduce a significant number of ops in the converted relay ir when there are a lot of boxes as input, it does happen in models like ssd in tensorflow object detection model zoo.
   
   @kevinthesun @zhiics @icemelon9 @jwfromm 
   

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services