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/06/03 23:25:58 UTC
[GitHub] [incubator-mxnet] Zha0q1 commented on a change in pull request
#15132: Profiler API Enhancements
Zha0q1 commented on a change in pull request #15132: Profiler API Enhancements
URL: https://github.com/apache/incubator-mxnet/pull/15132#discussion_r290075968
##########
File path: include/mxnet/c_api.h
##########
@@ -321,13 +321,18 @@ MXNET_DLL int MXDumpProcessProfile(int finished, int profile_process, KVStoreHan
MXNET_DLL int MXDumpProfile(int finished);
/*!
- * \brief Print aggregate stats to the a string
- * \param out_str Will receive a pointer to the output string
- * \param reset Clear the aggregate stats after printing
- * \return 0 when success, -1 when failure happens.
- * \note
- */
-MXNET_DLL int MXAggregateProfileStatsPrint(const char **out_str, int reset);
+ * \brief Print aggregate stats to the a string
+ * \param out_str Will receive a pointer to the output string
+ * \param reset Clear the aggregate stats after printing
+ * \param out_str will receive a pointer to the output string
+ * \param reset clear the aggregate stats after printing
+ * \param format whether to return in tabluar or json format
+ * \param sort_by sort by avg, min, max, or count
+ * \param ascending whether to sort ascendingly
+ * \return 0 when success, -1 when failure happens.
+ * \note
+ */
+MXNET_DLL int MXAggregateProfileStatsPrint(const char **out_str, int reset, int format, int sort_by, int ascending);
Review comment:
Yeah will do!
----------------------------------------------------------------
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