-
Notifications
You must be signed in to change notification settings - Fork 5.7k
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
[Bug fixes] Add default arg to enhance varbase ClearGradient func #36837
[Bug fixes] Add default arg to enhance varbase ClearGradient func #36837
Conversation
Thanks for your contribution! |
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.
LGTM for collective
Sorry to inform you that 9c458ff's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
…func to set/get gradient isEmpty
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.
some comments
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.
LGTM
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.
LGTM
PR types
Bug fixes
PR changes
APIs
Describe
1. Add arg to enhance varbase ClearGradient func,
use case: 1
use case: 2
2. Expose two func to set and get the " is_empty_ " attribute of grad_var_