reminisce opened a new pull request #8055: Skip adding control_deps if nodes 
are the same
URL: https://github.com/apache/incubator-mxnet/pull/8055
 
 
   The following code would fail before this PR due to introducing a cycle into 
the gradient graph. See this issue for analysis: 
https://github.com/apache/incubator-mxnet/issues/8029. This PR also fixes this 
issue of the same root cause: 
https://github.com/apache/incubator-mxnet/issues/7956.
   
   ```python
   data = mx.symbol.Variable('data')
   res = data + data + data + data + data + data + data + data
   res.simple_bind(ctx=mx.cpu(), data=(1,))
   ```
   
   @piiswrong @tqchen 
 
----------------------------------------------------------------
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

Reply via email to