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/08/16 18:14:57 UTC

[GitHub] [incubator-mxnet] Zha0q1 commented on issue #15132: Profiler API Enhancements

Zha0q1 commented on issue #15132: Profiler API Enhancements
URL: https://github.com/apache/incubator-mxnet/pull/15132#issuecomment-522103033
 
 
   > The `ascending` argument is great, but why is it missing the most important sort criteria "total time"? That's usually THE first thing people look at - what operator takes most of the time in a network. This is definitely a must-have for the 'sort-by' argument ..
   
   Sure. I will create a pr and add "total_time" soon. 

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