Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

support offload/reload optimizer's states for custom device #9467

Merged
merged 1 commit into from
Dec 2, 2024

Conversation

tianhaodongbd
Copy link
Contributor

PR types

Others

PR changes

Others

Description

support offload/reload optimizer's states for custom device

Copy link

paddle-bot bot commented Nov 20, 2024

Thanks for your contribution!

Copy link

codecov bot commented Nov 20, 2024

Codecov Report

Attention: Patch coverage is 30.00000% with 7 lines in your changes missing coverage. Please review.

Project coverage is 52.87%. Comparing base (921fc44) to head (6392c53).
Report is 59 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/trainer/trainer.py 30.00% 7 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9467      +/-   ##
===========================================
- Coverage    52.99%   52.87%   -0.12%     
===========================================
  Files          677      679       +2     
  Lines       109079   108348     -731     
===========================================
- Hits         57802    57291     -511     
+ Misses       51277    51057     -220     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@gongweibao gongweibao left a comment

Choose a reason for hiding this comment

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

LGTM

@wawltor wawltor merged commit eae8d9f into PaddlePaddle:develop Dec 2, 2024
9 of 12 checks passed
tianhaodongbd added a commit to tianhaodongbd/PaddleNLP that referenced this pull request Dec 9, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants