PaddleNLP icon indicating copy to clipboard operation
PaddleNLP copied to clipboard

Enable parallel_config to use commas as delimiters.

Open Difers opened this issue 1 year ago • 2 comments

PR types

Others

PR changes

Others

Description

  • https://github.com/PaddlePaddle/PaddleNLP/pull/8649 上述PR添加了","作为pipeline_parallel_config,sharing_parallel_config分隔符,但遗漏了一些处理,本PR对此做了补充。

Difers avatar Jun 28 '24 02:06 Difers

Thanks for your contribution!

paddle-bot[bot] avatar Jun 28 '24 02:06 paddle-bot[bot]

Codecov Report

Attention: Patch coverage is 25.00000% with 15 lines in your changes missing coverage. Please review.

Project coverage is 55.51%. Comparing base (5a508e5) to head (eb7cf45). Report is 217 commits behind head on develop.

Files with missing lines Patch % Lines
paddlenlp/trainer/training_args.py 9.09% 10 Missing :warning:
paddlenlp/trainer/trainer.py 44.44% 5 Missing :warning:
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8677      +/-   ##
===========================================
+ Coverage    55.43%   55.51%   +0.07%     
===========================================
  Files          626      626              
  Lines        98070    98064       -6     
===========================================
+ Hits         54366    54438      +72     
+ Misses       43704    43626      -78     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar Jun 28 '24 03:06 codecov[bot]