ArmageddonKnight commented on a change in pull request #17376: [MXNET-1404] 
Implement storage tagging, the first half of the memory profiler
URL: https://github.com/apache/incubator-mxnet/pull/17376#discussion_r368256685
 
 

 ##########
 File path: include/mxnet/c_api.h
 ##########
 @@ -313,6 +313,13 @@ MXNET_DLL int MXSetProcessProfilerState(int state, int 
profile_process,
  */
 MXNET_DLL int MXSetProfilerState(int state);
 
+/*!
+ * \brief Set the scope of profiler for current process
 
 Review comment:
   @ChaiBapchya As an example, for Gluon applications, the profiler scope is 
set to be the Gluon block name.

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