reminisce commented on issue #13143: [MXNET-1206] Support NDArray indexing with 
None and Ellipsis
URL: https://github.com/apache/incubator-mxnet/pull/13143#issuecomment-517894875
 
 
   @kohr-h Thanks for the reply. @zoeygxy did some benchmark and found that
   1. Indexing operations that should return views are slower than before.
   2. Indexing operations that should return newly allocated `NDArray`s are 
faster than before.
   
   We can get this PR merged first and take care of the cases that return views 
in a later PR.
   
   PS: benchmark results
   
[old_indexing.txt](https://github.com/apache/incubator-mxnet/files/3463396/old_indexing.txt)
   
[new_indexing.txt](https://github.com/apache/incubator-mxnet/files/3463397/new_indexing.txt)
   

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


With regards,
Apache Git Services

Reply via email to