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

[AutoParallel]:fix baichuan d2s fail #9478

Merged

Conversation

blacksheep-Aristotle
Copy link
Contributor

PR types

Bug fixes

PR changes

Models

Description

修复baichuan-14b动转静失败的bug

Copy link

paddle-bot bot commented Nov 22, 2024

Thanks for your contribution!

Copy link

codecov bot commented Nov 22, 2024

Codecov Report

Attention: Patch coverage is 20.00000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 53.10%. Comparing base (183e012) to head (d60fc77).
Report is 11 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/transformers/llama/modeling.py 25.00% 3 Missing ⚠️
paddlenlp/transformers/llama/modeling_auto.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9478      +/-   ##
===========================================
+ Coverage    52.98%   53.10%   +0.11%     
===========================================
  Files          685      685              
  Lines       108904   109552     +648     
===========================================
+ Hits         57702    58175     +473     
- Misses       51202    51377     +175     

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

Copy link
Collaborator

@wawltor wawltor 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 6141f80 into PaddlePaddle:develop Nov 26, 2024
8 of 12 checks passed
@blacksheep-Aristotle blacksheep-Aristotle deleted the fix_baichuan_auto branch December 19, 2024 06:43
# 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.

2 participants