Skip to content

Conversation

@Southpika
Copy link
Contributor

PR types

Bug fixes

PR changes

Others

Description

更新合并脚本适配pp,同时增加文件检查

@paddle-bot
Copy link

paddle-bot bot commented Apr 9, 2024

Thanks for your contribution!

parser.add_argument("--model_name_or_path", default=None, required=True, help="The directory of model.")
parser.add_argument("--device", type=str, default="gpu", help="Device")
parser.add_argument("--pipeline_parallel_degree", "--pp", type=int, required=True, help="pp degree")
parser.add_argument("--tensor_parallel_degree", "--tp", type=int, required=True, help="tp degree")
Copy link
Contributor

Choose a reason for hiding this comment

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

改一下readme

Copy link
Contributor

@lugimzzz lugimzzz left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Apr 9, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 55.09%. Comparing base (2273ee7) to head (2770ff5).
Report is 5 commits behind head on develop.

Additional details and impacted files
@@ Coverage Diff @@ ## develop #8239 +/- ## =========================================== - Coverage 55.15% 55.09% -0.06%  =========================================== Files 601 602 +1 Lines 91764 91861 +97 =========================================== + Hits 50611 50614 +3  - Misses 41153 41247 +94 

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

@wawltor wawltor merged commit c3ec984 into PaddlePaddle:develop Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants