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_r244627258
##########
File path: cpp-package/include/mxnet-cpp/monitor.h
##########
@@ -70,8 +70,9 @@ class Monitor {
/*!
* \brief install callback to executor. Supports installing to multiple
executors.
* \param exe The executor to install to.
+ * \param monitor_all If true, monitor both input and output, otherwise
monitor output only.
*/
- void install(Executor *exe);
+ void install(Executor *exe, bool monitor_all = false);
Review comment:
`\param monitor_all If true, monitor both input and output, otherwise
monitor output only.`
If it's not clear enough, please leave your questions here and I can add
more description.
----------------------------------------------------------------
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
