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 2018/08/20 16:45:28 UTC
[GitHub] sandeep-krishnamurthy closed pull request #12251: Tweak ExecType
descriptions
sandeep-krishnamurthy closed pull request #12251: Tweak ExecType descriptions
URL: https://github.com/apache/incubator-mxnet/pull/12251
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/include/mxnet/op_attr_types.h b/include/mxnet/op_attr_types.h
index 2bb2462d486..aa5d4e6de78 100644
--- a/include/mxnet/op_attr_types.h
+++ b/include/mxnet/op_attr_types.h
@@ -87,18 +87,19 @@ struct OpContext {
/*! \brief the execution type of the operator */
enum class ExecType {
- /*! \brief Forward/Backward are synchronize calls */
+ /*! \brief Forward/Backward are synchronous calls */
kSync,
/*!
- * \brief Forward/Backward are asynchronize,
+ * \brief Forward/Backward are asynchronous,
* will call OpContext.async_on_complete when operation finishes.
*/
kAsync,
/*!
- * \brief Cross device copy operation, this is a special operator
- * That indicates copy across devices, the input and output can sit on different device.
- * In current implementation, copy operator is specially handled by executor.
- * This flag is used for special case treatment and future extension of different copy ops.
+ * \brief Cross device copy operation, this is a special operator that indicates it will copy
+ * across devices. For example the input and output for this type of operator can potentially
+ * reside on different devices. In the current implementation, a copy operator is specially
+ * handled by an executor. This flag is used for special case treatment and future extension of
+ * different copy ops.
*/
kCrossDeviceCopy,
/*!
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services