-
Notifications
You must be signed in to change notification settings - Fork 63
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use runtime value of NumberProxy if it is produced by an operator (#231) #244
Conversation
for more information, see https://pre-commit.ci
Could you please explain in the issue why is it a problem and what do you want to achieve? |
@jjsjann123, with this PR your example script from #541 (comment) works! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kiya00, I have reopened this pull request because I think this is the correct fix for the bugs we have. I'm sorry that I missed it somehow.
Could you please add this example #541 (comment) as a test case.
Hi @IvanYashchuk , I have added the test, please take another look~ |
@t-vi, could you please merge this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @kiya00 @jjsjann123 @IvanYashchuk
What does this PR do?
Try to fix #231.
The changes in this PR go through all the NumberProxy in
saved_for_backward
, replace it with thevalue
in NumberProxy if it has no producer in the forward trace, otherwise keep the NumberProxy