[GitHub] anirudh2290 commented on issue #14123: Add int8 data loader

[GitBox <gi...@apache.org>]

anirudh2290 commented on issue #14123: Add int8 data loader
URL: https://github.com/apache/incubator-mxnet/pull/14123#issuecomment-468518573
 
 
   I guess there are good reasons for not allowing inplace operations for inputs. One reason could be that the data allocation for inputs is controlled by the user instead of engine and it should not be mutated. We can argue that identity operations should still be supported, but this means that the memory cannot be reused if identity is followed by an inplace operation. Also, the reference count for reused input memory will never drop to 0. This means that it cannot be recycled and reused in the computation graph. 
   
   According to me, even if we cannot do inplace optimization we should still add support in the operator for int8 to make the user experience easier. 

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