已合并
skip sub-comm creation when global process group was reinit'd and fix p2p longname err #38993
limuan创建于 6月23日
skip sub-comm creation when global process group was reinit'd and fix p2p longname err #38993
已合并
limuan成员
6月23日
6月23日
# 【合入来源】
- [ ] 需求
- [x] 问题单
- [ ] issue/工单
- [ ] 重构优化
- [ ] 资料更新
# 【修改方案】
HCCLUdi在rootinfo场景下的P2P通信中,被覆盖为groupname,导致hccl算子拼接后hcclUdi和hcclCommname后,字符超长,拼接后名字:groupbfa08d29a349dccf55aaf7ea17c96695164d8ba6_p2p_0_1_groupbfa08d29a349dccf55aaf7ea17c96695164d8ba6_p2p_0_1
修复方案:Udi仅在rankfiletable场景下,被覆盖为groupname
# 【资料变更】
不涉及
# 【接口变更】
不涉及
# 【功能验证】
ut用例通过:
pp并行时,正常拉起训练
Udi正常更新:
# 【CheckList】
> PR提交人对以下CheckList自检项进行全量自检，自检通过或不涉及，均修改 [ ] 为 [x]
- [ ] 代码注释完备，正确记录错误日志
- [ ] 代码实现进行了返回值、空指针等校验
- [ ] PR标题正确使用类型标签，如：feat、fix、refactor、docs、test等
- [ ] PR持续集成流水线（CI）执行通过，代码检查无异常
Pull Request已成功合入, 合并人@ascend-robot
（感谢 limuan 的贡献）
ascend-robot
6月23日 评论：
6月23日 评论：
6月23日 添加了label：ascend-cla/yes
ascend-robot
6月23日 评论：
6月23日 评论：
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 |
|---|---|---|
| test | ✅ renyujin, pengjingyou (2/2) | ✅ renyujin (1/1) |
| torch_npu/csrc/distributed | ✅ renyujin, pengjingyou (2/2) | ✅ renyujin (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
limuan, thanks for your pull request. All authors of the commits have signed the CLA. 👍
6月23日 添加了label：needs-issue
此处折叠了89条消息 查看更多
pengjingyou
6月23日 评论：
6月23日 评论：
/lgtm
6月23日 添加了label：approvedlgtm
6月23日 合入了pull request
ascend-robot
6月23日 评论：
6月23日 评论：
流水线 pytorch_gitcode_PR_multiVersion#11147 [ commitID：0cb1ba70 ] 已完成