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 2019/06/14 02:02:16 UTC

[GitHub] [incubator-mxnet] junrushao1994 edited a comment on issue #14192: [MXNET-1324] Add NaiveRunGraph to imperative utils

junrushao1994 edited a comment on issue #14192: [MXNET-1324] Add NaiveRunGraph to imperative utils
URL: https://github.com/apache/incubator-mxnet/pull/14192#issuecomment-501937227
 
 
   > Since this commit might cause a regression, I would kindly request if you can elaborate a bit about the differences between running operators one at a time vs the previous behaviour.
   
   I agree. This is part of [this](https://cwiki.apache.org/confluence/display/MXNET/Dynamic+shape) proposal by @zheng-da, but I believe it will be super helpful if
   1) the changes are reflected or elaborated in our docs so that the behavior is clearly understood.
   2) suppress the newly added naive run graph pass if other flags (like static_alloc) are on.

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


With regards,
Apache Git Services