Skip to content
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

[Fix] Clear differentiation cache in visu_forward to avoid OOM problem #757

Merged

Conversation

HydrogenSulfate
Copy link
Collaborator

PR types

Bug fixes

PR changes

APIs

Describe

  1. visu_forward 结尾固定清除微分缓存,否则如果expr_dict为None而微分操作注册到了model内,则可能会导致显存缓慢增大最终溢出

Copy link

paddle-bot bot commented Jan 17, 2024

Thanks for your contribution!

Copy link
Collaborator

@zhiminzhang0830 zhiminzhang0830 left a comment

Choose a reason for hiding this comment

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

LGTM

@zhiminzhang0830 zhiminzhang0830 merged commit 6ca0583 into PaddlePaddle:develop Jan 18, 2024
3 checks passed
@HydrogenSulfate HydrogenSulfate deleted the fix_oom_for_bubble branch January 28, 2024 08:45
huohuohuohuohuo123 pushed a commit to huohuohuohuohuo123/PaddleScience that referenced this pull request Aug 12, 2024
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