Skip to content

Conversation

@ForFishes
Copy link
Member

PR types

Bug fixes

PR changes

Others

Describe

Fix traverse graph bug in reducer.
During the forward execution, the graph will be traversed to find the unused parameter. Because the intermediate variable clerar of the graph has been changed in the previous reverse direction, some of the checker with input and output failed.

@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

Copy link

@sandyhouse sandyhouse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ForFishes ForFishes merged commit f4a3f85 into PaddlePaddle:develop Apr 30, 2021
@ForFishes ForFishes deleted the fix_reducer_prepare_backward branch April 30, 2021 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants