已合并
fix: Disallow use-global-aux-loss when moe-alltoall-overlap-comm is enabled #4389
zzyyjj012创建于 4月8日
fix: Disallow use-global-aux-loss when moe-alltoall-overlap-comm is enabled #4389
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 zzyyjj012 的贡献)ascend-robot
4月8日 评论:
4月8日 评论:
ascend-robot
4月8日 评论:
4月8日 评论:
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 | ✅ 郭志华, yanzhixiao23 (2/2) | ✅ 郭志华 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
zzyyjj012, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月8日 添加了label:ascend-cla/yes
AtlasAccount
4月8日 评论:
4月8日 评论:
已进入队列,请勿重复触发


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


4月8日 添加了label:docs-ci-pipeline-running
AtlasAccount
4月8日 评论:
4月8日 评论:
预计 2026-04-08 16:10:49 开始,等待时长:19.00min,排队位置:1


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


ascend-robot
4月8日 评论:
4月8日 评论:
4月8日 删除了label:docs-ci-pipeline-running
4月8日 添加了label:docs-ci-pipeline-success
ascend-robot
4月8日 评论:
4月8日 评论:
4月8日 添加了label:ci-pipeline-running
4月8日 删除了label:ci-pipeline-running
4月8日 添加了label:ci-pipeline-passed
ascend-robot
4月8日 评论:
4月8日 评论:
yanzhixiao
4月9日 评论:
4月9日 评论:
/lgtm


guozhihua2
4月9日 评论:
4月9日 评论:
/approve


4月9日 添加了label:approvedlgtm
ascend-robot
4月9日 评论:
4月9日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: 郭志华.
Reviewers who wrote a comment of /lgtm are: 郭志华, yanzhixiao23.


4月9日 合入了pull request
What this PR does / why we need it?
当use-global-aux-loss和moe-alltoall-overlap-comm同时开启时,显式抛出异常。
Does this PR introduce any user-facing change?
无
How was this patch tested?
关闭moe-alltoall-overlap-comm时,use-global-aux-loss可正常开启使用。打开moe-alltoall-overlap-comm时,抛出异常。