已合并
[test]fix _test_alltoall_2p_size test_mgr.load #32520
AACAES创建于 3月27日
[test]fix _test_alltoall_2p_size test_mgr.load #32520
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 AACAES 的贡献）
AtlasAccount
3月27日 评论：
3月27日 评论：
ascend-robot
3月27日 评论：
3月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/pytorch | ✅ htchu, li_jing_hw (2/2) | ✅ htchu, li_jing_hw (2/1) |
| test | ✅ li_jing_hw, htchu, yuhaiyan (3/2) | ✅ li_jing_hw, htchu (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
AACAES, thanks for your pull request. All authors of the commits have signed the CLA. 👍
3月27日 添加了label：ascend-cla/yes
AtlasAccount
3月27日 评论：
3月27日 评论：
问题/功能描述
本次PR主要包含两项变更：一是移除了测试文件加载逻辑中针对分布式和inductor测试的过滤条件，确保在指定world_size时相关测试文件仍能被正常加载和运行；二是在一个特定的分布式通信测试用例中，启用了NPU内部格式张量的支持，以确保测试能够正确执行涉及特殊存储格式的张量操作。
修改方案描述
- 测试管理逻辑调整：在
TestMgr.load方法中，删除了当world_size不为零时跳过包含”test/distributed/”或”test/_inductor/”路径的修改文件的逻辑。这使得测试管理器在分布式或多进程场景下，也能将这些路径下的测试文件纳入分析范围。 - 测试用例功能增强：在
test/distributed/test_all_to_all.py文件的_test_alltoall_2p_size方法开头，添加了torch_npu.npu.config.allow_internal_format = True配置。这允许在该测试中使用NPU的内部存储格式（format 29），从而验证分布式通信操作（all_to_all）对内部格式张量的支持情况。
此处折叠了6条事件消息 查看更多
3月27日 添加了label：ci-pipeline-passed
ascend-robot
3月27日 评论：
3月27日 评论：
流水线 PR-pipeline_pytorch#12889 （重试第2次）已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| Build_ARM_inductor | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_01 | ✅ | >>> | |
| UT_DIST_X86_Part_02 | ✅ | >>> | |
| UT_DIST_X86_Part_03 | ✅ | >>> | |
| UT_DIST_X86_Part_04 | ✅ | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |
haiyan8
3月28日 评论：
3月28日 评论：
/lgtm
li_jing_hw
3月28日 评论：
3月28日 评论：
/approve
3月28日 添加了label：approved
3月30日 添加了label：lgtm
ascend-robot
3月30日 评论：
3月30日 评论：
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: htchu, li_jing_hw.
Reviewers who wrote a comment of /lgtm are: yuhaiyan, li_jing_hw, htchu.
3月30日 合入了pull request
【合入来源】
【修改方案】
allow_internal_format默认值被改为False,_test_alltoall_2p_size 用例需手动设为True
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
本pr只修一个分布式用例，ci跑到了
【CheckList】