Skip to content

Conversation

@zhangyuqin1998
Copy link
Contributor

@zhangyuqin1998 zhangyuqin1998 commented Sep 22, 2025

PR types

Function optimization

PR changes

Others

Description

Pass a local function to forward_backward_pipeline instead of the dataset itself. This prevents the dataset from being passed as a direct argument to forward_backward_pipeline, which would create additional reference counts that cannot be cleared, leading to GPU memory leaks.

This PR depends on PaddlePaddle/Paddle#75446.

@paddle-bot
Copy link

paddle-bot bot commented Sep 22, 2025

Thanks for your contribution!

@zhangyuqin1998
Copy link
Contributor Author

/rerun all-failed

1 similar comment
@zhangyuqin1998
Copy link
Contributor Author

/rerun all-failed

Copy link
Member

@ForFishes ForFishes left a comment

Choose a reason for hiding this comment

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

LGTM

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.

3 participants