Zha0q1 commented on issue #15069: [WIP] Profiler RFC: Introducing new APIs
URL: 
https://github.com/apache/incubator-mxnet/issues/15069#issuecomment-496783841
 
 
   > > Why add a new profiler API get_summary() in the back-end rather than a 
python parser utility that returns in JSON?
   > > This is we can use this new API in different languages and make sure the 
return is consistent.
   > 
   > I wasn't able to understand the answer for this question. @ZhaoqiZhu would 
you mind elaborate? I had the impression that there's no difference in the 
profiler dump file regardless of the frontend so I assumed that the later steps 
on analyzing the dump don't involve the front-end language anymore.
   
   Yes, the dump file is processed by the backend, but that's more for tracing. 
For aggregate information we currently only have dumps() which will return a 
tabular string. It prints out like this.
   
   
![](https://user-images.githubusercontent.com/10722037/58362190-49f4c080-7e49-11e9-92a3-23664384544b.png)
   
   So instead of adding a utility that parse that table into a JSON string, we 
can return a JSON string from the backend directly.

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


With regards,
Apache Git Services

Reply via email to