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

fixed the init problem in tensor parallel #9452

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

wawltor
Copy link
Collaborator

@wawltor wawltor commented Nov 19, 2024

PR types

Bug fixes

PR changes

Models

Description

fixed the init problem in tensor parallel

Copy link

paddle-bot bot commented Nov 19, 2024

Thanks for your contribution!

Copy link

codecov bot commented Nov 19, 2024

Codecov Report

Attention: Patch coverage is 14.28571% with 6 lines in your changes missing coverage. Please review.

Project coverage is 52.93%. Comparing base (81a2d43) to head (a0a173a).
Report is 21 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/ops/distributed/parallel.py 16.66% 5 Missing ⚠️
paddlenlp/trainer/trainer.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #9452      +/-   ##
===========================================
- Coverage    52.96%   52.93%   -0.03%     
===========================================
  Files          681      676       -5     
  Lines       108066   107933     -133     
===========================================
- Hits         57238    57136     -102     
+ Misses       50828    50797      -31     

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


🚨 Try these New Features:

Copy link
Collaborator

@DrownFish19 DrownFish19 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 5a343cf into PaddlePaddle:develop Nov 21, 2024
9 of 12 checks passed
# 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