sandeep-krishnamurthy commented on issue #13837: Image ToTensor operator - GPU 
support, 3D/4D inputs
URL: https://github.com/apache/incubator-mxnet/pull/13837#issuecomment-455362969
 
 
   @szha 
   As commented in other PR - 
https://github.com/apache/incubator-mxnet/pull/13802
   1. I did not mean it is not an operator. I meant in this PR we reorganize, 
cpu/gpu support, and more such properties of an operator are added. Reworded 
the description.
   2. IMO, the reorganizing and creation of new files for this operator is in 
the right direction and unifies with other operator pattern. Having all these 
implementation in image_random-inl.h creates more confusion and out of order 
compared to other operators.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to