已合并
fix(pytorch): fix deepseekv4 script tp #4431
sunjunjie1587创建于 4月27日
fix(pytorch): fix deepseekv4 script tp #4431
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 sunjunjie1587 的贡献)4月27日 创建了 pull request,commit 1640378c
ascend-robot
4月27日 评论:
4月27日 评论:
ascend-robot
4月27日 评论:
4月27日 评论:
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/MindSpeed-LLM | ✅ 丁子霖, qyzqyz, HANHU1CHEN (3/2) | ✅ 丁子霖 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
sunjunjie1587, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月27日 添加了label:ascend-cla/yes
ascend-robot
4月27日 评论:
4月27日 评论:
Ascend docs pipeline is running...


4月27日 添加了label:docs-ci-pipeline-running
ascend-robot
4月27日 评论:
4月27日 评论:
✅ 跳过 docs ci 检查,没有需要检查的文档文件


4月27日 删除了label:docs-ci-pipeline-running
4月27日 添加了label:docs-ci-pipeline-success
sunjunjie1587
4月27日 评论:
4月27日 评论:
compile


AtlasAccount
4月27日 评论:
4月27日 评论:
已进入队列,请勿重复触发


4月27日 添加了label:ci-pipeline-running
AtlasAccount
4月27日 评论:
4月27日 评论:
正在运行中,预计 2026-04-27 11:35:17 运行结束,具体时间已实际情况为准,请勿重复compile


ascend-robot
4月27日 评论:
4月27日 评论:
Ascend docs pipeline is running...


4月27日 删除了label:docs-ci-pipeline-success
4月27日 添加了label:docs-ci-pipeline-running
ascend-robot
4月27日 评论:
4月27日 评论:
✅ 跳过 docs ci 检查,没有需要检查的文档文件


4月27日 删除了label:docs-ci-pipeline-running
4月27日 添加了label:docs-ci-pipeline-success
4月27日 删除了label:ci-pipeline-running
4月27日 添加了label:ci-pipeline-passed
ascend-robot
4月27日 评论:
4月27日 评论:
温一盏
4月27日 评论:
4月27日 评论:
/lgtm


HANHU1CHEN
4月27日 评论:
4月27日 评论:
/lgtm


4月27日 添加了label:lgtm
4月27日 添加了label:approved
ascend-robot
4月27日 评论:
4月27日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 丁子霖.
Reviewers who wrote a comment of /lgtm are: 丁子霖, qyzqyz, HANHU1CHEN.


4月27日 合入了pull request
What this PR does / why we need it?
Update the dsv4 script: change tp parameter from 2 to 1 due to a bug in the weight conversion where the etp parameter does not take effect.
Does this PR introduce any user-facing change?
change tp parameter from 2 to 1
How was this patch tested?
The new parameters have been verified at the customer site.