ZhennanQin commented on a change in pull request #13697: [MKLDNN] Enable signed
int8 support for convolution.
URL: https://github.com/apache/incubator-mxnet/pull/13697#discussion_r247388108
##########
File path: include/mxnet/c_api.h
##########
@@ -1833,10 +1832,12 @@ MXNET_DLL int
MXExecutorGetOptimizedSymbol(ExecutorHandle handle,
/*!
* \brief set a call back to notify the completion of operation
+ * \param monitor_all If true, monitor both input and output, otherwise
monitor output only.
*/
MXNET_DLL int MXExecutorSetMonitorCallback(ExecutorHandle handle,
ExecutorMonitorCallback callback,
- void* callback_handle);
+ void* callback_handle,
+ bool monitor_all);
Review comment:
According to
https://stackoverflow.com/questions/1472138/c-default-arguments, we shouldn't
add default value for this file as it defined C api instead of C++. So I
removed the default value.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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
