TaoLv commented on a change in pull request #11251: [WIP] Graph partitioner and 
subgraph op
URL: https://github.com/apache/incubator-mxnet/pull/11251#discussion_r195468081
 
 

 ##########
 File path: include/mxnet/op_attr_types.h
 ##########
 @@ -98,7 +98,12 @@ enum class ExecType {
    *  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.
    */
-  kCrossDeviceCopy
+  kCrossDeviceCopy,
+  /*!
+   * A subgraph execution should happen in the main thread, instead of
+   * in the execution engine.
 
 Review comment:
   operators inside a subgraph are still executed in the execution engine, 
right?

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to