Star850
1331
代码
代码
Issues1517
Pull Requests2239
流水线
项目讨论
Wiki
分析
Star850
1331
已合并
test(npu): add pytree supported nodes pop coverage #41810
jerry lee创建于 7月16日
test(npu): add pytree supported nodes pop coverage #41810
已合并
jerry lee创建于 7月16日
从whirlpool_dark/pytorchtest-pytree-supported-nodes-pop-v2.7.1合入到Ascend/pytorchv2.7.1
jerry lee
jerry lee
7月16日

【合入来源】

[Usage]: torch.utils._pytree.SUPPORTED_NODES.pop 缺少跨版本的直接 NPU 用例

【修改方案】

上游 v2.7.1 的 torch.onnx._internal.fx.dynamo_graph_extractor._PyTreeExtensionContext.__exit__ 调用 pop(class_type)，但没有直接测试；v2.9.0、v2.10.0 没有直接调用；v2.11.0、v2.12.0 仅在注册表清理中调用 pop(cls, None)，未验证返回值和删除后的展开语义。因此新增 test/test_pytree_registry.py。

test.test_pytree_registry.TestPytreeSupportedNodes.test_pop 的调用路径为：

torch.utils._pytree.register_pytree_node(CustomNode, ...)
-> torch.utils._pytree.tree_flatten(CustomNode(npu_tensor))
-> torch.utils._pytree.SUPPORTED_NODES[CustomNode].flatten_fn
-> torch.utils._pytree.SUPPORTED_NODES.pop(CustomNode)
-> torch.utils._pytree.SUPPORTED_NODES.pop(CustomNode, None)
-> torch.utils._pytree.tree_flatten(CustomNode(npu_tensor))

用例验证无默认值的 pop(CustomNode) 返回原 NodeDef 的 flatten_fn 并删除注册项，带默认值的 pop(CustomNode, None) 返回 None，删除后同一节点回退为单个叶子。Tensor 使用 torch.tensor([1.0]).to(device_type)，其中 device_type 从当前 accelerator 获取；清理通过 TestCase.addCleanup() 恢复注册项后再注销，避免污染后续测试，不捕获 API 异常。

【资料变更】

torch.utils._pytree.SUPPORTED_NODES.pop 位于 PyTorch 私有模块，不需要更新 native API 支持文档。

【接口变更】

不涉及 API 源码或用户接口，仅新增 torch-npu 测试。

【功能验证】

python3 -m py_compile test/test_pytree_registry.py 和 git diff --check 已通过。为避免 torch-npu 源码 checkout 遮蔽 pip 安装包，从中立的 /workspace 目录执行同一测试文件，NPU 云算力日志如下：

torch=2.7.1+cpu
torch_npu=2.7.1.post6
accelerator=npu
[root@73c35cf5cac2 workspace]# python3 ./test_pytree_registry_2.7.1.py TestPytreeSupportedNodes.test_pop
.
----------------------------------------------------------------------
Ran 1 test in 1.019s

OK

【CheckList】

likedislike
Pull Request已成功合入, 合并人@ascend-robot
（感谢 jerry lee 的贡献）
jerry leejerry lee
7月16日 创建了 pull request，commit 8e1f3042
jerry leejerry lee
7月16日 关联了issue：[Usage]: `torch.utils._pytree.SUPPORTED_NODES.pop` 缺少跨版本的直接 NPU 用例
atomgit-bot
atomgit-bot
7月16日 评论：

变更摘要

此 PR 新增了 test/test_pytree_registry.py 测试文件，专门用于验证 torch.utils._pytree.SUPPORTED_NODES.pop 在 NPU 设备上的行为。用例覆盖了注册自定义 pytree 节点后，通过 pop 删除注册项并验证返回值、删除后的回退语义（节点退化为叶子），以及 tree_flatten 在不同阶段的行为一致性。设备类型通过 torch.accelerator.current_accelerator() 动态获取，默认回退到 CPU。

主要改动

  • 新增 test/test_pytree_registry.py 测试文件：包含 TestPytreeSupportedNodes 测试类及 test_pop 方法，完整覆盖 SUPPORTED_NODES.pop 的两种调用方式（带默认值与不带默认值）及其副作用。
  • 验证 pop(CustomNode) 无默认值行为：断言返回的 node_def.flatten_fn 与注册时的 flatten_fn 一致，且 CustomNode 已从 SUPPORTED_NODES 中移除。
  • 验证 pop(CustomNode, None) 带默认值行为：在节点已被删除后再次调用，断言返回 None。
  • 验证删除后的回退语义：注册项被 pop 移除后，再次调用 tree_flatten 时节点被视为单个叶子（spec.is_leaf() 为 True），且返回的 leaves 直接为 [node] 本身。
  • 设备类型动态适配：使用 torch.accelerator.current_accelerator() 获取当前加速器类型，若无加速器则回退为 "cpu"，使测试能在 NPU 和 CPU 环境下运行。
likedislike
ascend-robot
ascend-robot成员
7月16日 评论：
atomgit-bot
atomgit-bot
7月16日 评论：

代码审查

✅ 未发现问题

likedislike
ascend-robotascend-robot成员
7月16日 添加了label：ascend-cla/yes
ascend-robot
ascend-robot成员
7月16日 评论：

CLA Signature Pass

whirlpool_dark, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
ascend-robot
ascend-robot成员
7月16日 评论：

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 ✅ liangsongwei, huangjingwei (2/2) ✅ liangsongwei (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
ascend-robot
ascend-robot成员
7月16日 评论：

当前仓库存在以下 保护分支 ：

Protected Branch Version Release
master
v2.10.0
v2.12.0
v2.7.1
v2.9.0
v2.11.0
v2.12.0-26.1.0
v2.10.0-26.1.0
v2.9.0-26.1.0
v2.7.1-26.1.0
v2.11.0-26.1.0
ci-test

评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作

注意：

  1. /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
  2. 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭
likedislike
ascend-robotascend-robot成员
7月16日 添加了label：ci-pipeline-running
ascend-robot
ascend-robot成员
7月16日 评论：

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
7月16日 添加了label：docs-ci-pipeline-running
ascend-robot
ascend-robot成员
7月16日 评论：

✅ 跳过 docs ci 检查，没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
7月16日 删除了label：docs-ci-pipeline-running
ascend-robotascend-robot成员
7月16日 添加了label：docs-ci-pipeline-success
ascend-robotascend-robot成员
7月16日 删除了label：ci-pipeline-running
ascend-robotascend-robot成员
7月16日 添加了label：ci-pipeline-passed
ascend-robot
ascend-robot成员
7月16日 评论：
流水线 PR-pipeline_pytorch#45865 [ commitID：a6760c3b ] 已完成
>>>代码风格自动修复执行成功（无修复内容）
阶段 任务名 状态 详情
编译构建 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 ✅ >>>
此流水线已支持下列评论快捷指令，仅PR创建者和白名单成员[wujinyuan1, huangjingwei, liangsongwei, yashi999, culechan, Dring, wuyouqi1, L1919_snow, qq_52711437, WhiteNight12, nomiz, xiu_21, ffmh, wanglijun55, hss-shuai, husichao, smallsilly, lanshaozuishuai, jimmyisme1, lzy0920232, alpha-junh, Sunshine_Youngster, wei_zhuoyi, zhangyihuiben, zyw-hw, zzzkeke, rmch, yangch0324, LucciC, AACAES, renyujin, wjlflyer, senzhen-town, pengjingyou, qsc97, limuan, yule100, xiaoqi-zhou, kuhn7, chenxingying, hanye02, zichun_ye, anyrenwei, kkjocker, wangzili121, Lu_G, yvjc, puddingfjz, HandsoemLemon, bigprestigee1, huawuyi, zhenyu10, dairenjie, du-jin-hang, zou-jieyu, adelaideliu, TrHan, wanlinan, Windwindzzz, pengqihw, kisnwang, yuheng_wang, honghao_wang, jizewei, zhangguoguang, sunyu-xuan, chenrayray, hbhu_bin, liujunzhu, c_34, LiNuoh, maoyuanpeng1, zzhongmin, zhaoyu65, bellatan, jiabaolin, zhuofanshen, wencaiwen, lu_zhuge, caoshuyang, molly12, lyx324521, LQ1206, gitcode-chenjiao, cai-weiwei1989, CHDong, ogqin, yuanlipingGit, xuqinglin1, lqz2, zouwei1, chaoluoa, paradox325, jackzhang1116, yaoyao, akh, yujiacheng, dengjie0116, Hubert11111, Shine_Ws, wslhj555, longqiand, OYtao666, JiaqingQiang, luyyyy, Kingbelial, zhanghaiyu0101, wenxp1018, yanliu-luoluo, ksun_sekiro, liyong328, wgzheng, tangky, vivi_is_coding, aoiaoisola, weixin_44494597, wangmengmengwang65667, hid57809721, qq_35468730, comeonup, C547032, gcw_m5OQChA4, yao_yao_ling_xian, cnnbwcy, szqfes_12, cora_19, cann_lilin, can, shawnylee233, fanglanyue0916, hhz0, LiNuohang, taohuoquan]评论有效
  • compile、compile_inductor、compile_torchair : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
Jingwei Huang
Jingwei Huang成员
7月22日 评论：

/lgtm

likedislike
梁松伟
梁松伟成员
7月23日 评论：

/approve

likedislike
ascend-robotascend-robot成员
7月23日 添加了label：approvedlgtm
ascend-robotascend-robot成员
7月23日 删除了label：ci-pipeline-passed
ascend-robotascend-robot成员
7月23日 合入了pull request
ascend-robot
ascend-robot成员
7月23日 评论：
流水线 pytorch_gitcode_PR_multiVersion#13006 [ commitID：a6760c3b ] 已完成
likedislike