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

Update merge_lora_params.py #8514

Merged
merged 3 commits into from
May 31, 2024
Merged

Conversation

Galaxy1458
Copy link
Contributor

@Galaxy1458 Galaxy1458 commented May 30, 2024

PR types

PR changes

Description

修复lora_merge bug

Copy link

paddle-bot bot commented May 30, 2024

Thanks for your contribution!

Copy link

codecov bot commented May 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 54.03%. Comparing base (92aa9f1) to head (8372c95).

Current head 8372c95 differs from pull request most recent head 6c1584a

Please upload reports for the commit 6c1584a to get more accurate results.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8514      +/-   ##
===========================================
- Coverage    54.03%   54.03%   -0.01%     
===========================================
  Files          618      618              
  Lines        96697    96697              
===========================================
- Hits         52252    52249       -3     
- Misses       44445    44448       +3     

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

ZHUI
ZHUI previously approved these changes May 30, 2024
Copy link
Collaborator

@ZHUI ZHUI 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 60c6615 into PaddlePaddle:develop May 31, 2024
7 of 10 checks passed
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