已合并
fix(test): correct expected buffer size calculation in test_hccl_shared_buffer #39515
limuan创建于 6月29日
fix(test): correct expected buffer size calculation in test_hccl_shared_buffer #39515
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 limuan 的贡献）
ascend-robot
6月29日 评论：
6月29日 评论：
6月29日 添加了label：ascend-cla/yes
ascend-robot
6月29日 评论：
6月29日 评论：
6月29日 添加了label：needs-issue
ascend-robot
6月29日 评论：
6月29日 评论：
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, senzhen-town (2/2) | ✅ renyujin (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
ascend-robot
6月29日 评论：
6月29日 评论：
Linking Issue Notice
@limuan , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .
ascend-robot
6月29日 评论：
6月29日 评论：
当前仓库存在以下 保护分支 ：
| Protected Branch | Version | Release |
|---|---|---|
| master | ||
| v2.9.0 | ||
| v2.9.0-26.1.0 | ||
| v2.7.1 | ||
| v2.7.1-26.1.0 | ||
| v2.10.0-26.1.0 | ||
| v2.11.0-26.1.0 | ||
| v2.10.0 | ||
| v2.12.0 | ||
| v2.11.0 | ||
| v2.12.0-26.1.0 | ||
| ci-test | ||
| sync-pr28113--to-v2.9.0 |
评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作
注意：
- /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
- 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭
6月29日 添加了label：ci-pipeline-running
atomgit-bot
6月29日 评论：
6月29日 评论：
变更摘要
此 PR 将 test_hccl_shared_buffer.py 中的内存预期值计算公式从 buffer_size * num_different_buffer_group + tol 修正为 (buffer_size + tol) * num_different_buffer_group，使容差 tol 按每个 buffer group 分摊，而非仅在乘积结果上一次性加上。该改动统一应用于创建 pg1、pg2、pg3 三处相同模式的测试代码，确保测试对 HCCL 共享缓冲区内存占用的预期计算更加合理。
主要改动
- 修正
expected计算公式的运算符优先级: 将expected = buffer_size * num_different_buffer_group + tol改为expected = (buffer_size + tol) * num_different_buffer_group，使容差tol与buffer_size先相加再乘以 group 数量，从而让每个不同的 buffer group 都能获得独立的容差预算。 - pg1 内存预期值修正: 在 pg1 创建后将
expected的计算统一为新公式（第 316 行）。 - pg2 内存预期值修正: 在 pg2（与 pg1 共享同一 buffer）创建后同步更新
expected计算公式（第 328 行）。 - pg3 内存预期值修正: 在 pg3（使用不同 buffer
subSharedBuffer_2）创建后同步更新expected计算公式（第 341 行）。
ascend-robot
6月29日 评论：
6月29日 评论：
ascend docs pipeline is running...
6月29日 添加了label：docs-ci-pipeline-running
ascend-robot
6月29日 评论：
6月29日 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
6月29日 删除了label：docs-ci-pipeline-running
6月29日 添加了label：docs-ci-pipeline-success
atomgit-bot
6月29日 评论：
6月29日 评论：
代码审查
审查总结
本次 diff 仅涉及 test/distributed/test_hccl_shared_buffer.py 一个文件，将 _test_buffer_memory 方法中的三处 expected 计算公式从 buffer_size * num_different_buffer_group + tol 改为 (buffer_size + tol) * num_different_buffer_group。
- 已审查文件：
test/distributed/test_hccl_shared_buffer.py— 发现 2 个问题 - P3 问题：2 个
- 整体风险评估：低。这是测试文件的改动，不影响生产代码。两个问题均为测试质量相关：容差语义与公式不匹配可能弱化内存回归检测能力，以及兄弟测试方法间的公式不一致。建议作者确认
tol的真实语义后统一处理。
⚠️ 已识别出整体风险，但无法提取行内评论，请参考整体评估。
6月29日 删除了label：ci-pipeline-running
6月29日 添加了label：ci-pipeline-passed
ascend-robot
6月29日 评论：
6月29日 评论：
流水线 PR-pipeline_pytorch#39796 [ commitID：b4ef297f ] 已完成
>>>代码风格自动修复执行成功（无修复内容）
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| CodeCheck_lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| UT_ARM_A2_Part_03 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_DIST_ARM_Part_01 | ✅ | >>> | |
| UT_DIST_ARM_Part_02 | ✅ | >>> | |
| UT_DIST_ARM_Part_03 | ✅ | >>> | |
| UT_DIST_ARM_Part_04 | ✅ | >>> | |
| UT_ARM_A2_Select_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Select_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
- compile、compile_inductor、compile_torchair : 运行流水线
- retry : 重试流水线所有失败子任务
- retry <任务名> : 仅重试指定失败子任务
- stop : 停止流水线
limuan6月29日 修改标题为 “fix(test): correct expected buffer size calculation in test_hccl_shared_buffer”，原标题为“Apply test_hccl_shared_buffer.py changes to hccl_shared_ut_v2.10.0”
6月29日 修改标题为 “fix(test): correct expected buffer size calculation in test_hccl_shared_buffer”，原标题为“Apply test_hccl_shared_buffer.py changes to hccl_shared_ut_v2.10.0”
6月29日 修改了pull request 的描述
limuan
7月7日 评论：
7月7日 评论：
compile
7月7日 删除了label：ci-pipeline-passed
7月7日 添加了label：ci-pipeline-running
ascend-robot
7月7日 评论：
7月7日 评论：
ascend docs pipeline is running...
7月7日 删除了label：docs-ci-pipeline-success
7月7日 添加了label：docs-ci-pipeline-running
ascend-robot
7月7日 评论：
7月7日 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
7月7日 删除了label：docs-ci-pipeline-running
7月7日 添加了label：docs-ci-pipeline-success
7月7日 删除了label：ci-pipeline-running
7月7日 添加了label：ci-pipeline-passed
ascend-robot
7月7日 评论：
7月7日 评论：
流水线 PR-pipeline_pytorch#41753 [ commitID：b4ef297f ] 已完成
>>>代码风格自动修复执行成功（无修复内容）
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| CodeCheck_lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| UT_ARM_A2_Part_03 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_DIST_ARM_Part_01 | ✅ | >>> | |
| UT_DIST_ARM_Part_02 | ✅ | >>> | |
| UT_DIST_ARM_Part_03 | ✅ | >>> | |
| UT_DIST_ARM_Part_04 | ✅ | >>> | |
| UT_ARM_A2_Select_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Select_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
- compile、compile_inductor、compile_torchair : 运行流水线
- retry : 重试流水线所有失败子任务
- retry <任务名> : 仅重试指定失败子任务
- stop : 停止流水线
7月7日 添加了label：approved
senzhen-town
7月7日 评论：
7月7日 评论：
/lgtm
7月7日 关联了issue：HCCL 通信域创建占用内存,A2,A3存在差异,需分析
7月7日 删除了label：needs-issue
7月7日 添加了label：lgtm
7月7日 合入了pull request
【合入来源】
【修改方案】
用例中,原来内存占用的预期值为:groupnum*buffer_size + tol (tol为100,在通信域较多,可能无法coverbuffersize外的通信域内存开销),现在改为
每个通信域多预留100MB，100MB远远小于用例本身buffersize的400MB,可以看护共享内存功能
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
修改后用例通过
【CheckList】