arcadiaphy edited a comment on issue #15574: fix naive engine for 
multi-threaded inference
URL: https://github.com/apache/incubator-mxnet/pull/15574#issuecomment-513525766
 
 
   @anirudh2290 Yes, in order to prevent async call from being pushed, using a 
single req_completed_ is not enough. We can add the req_completed_ to NaiveOpr 
and check whether it's been set to true in complete callback.
   
   Edit:
   I used a simpler implementation to remove field req_completed_. Since cpu 
stream is dummy stream, so for cpu mode, the naive engine is now stateless not 
considering bool flag shutdown_phase_.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to