已合并
fix test_autograd to avoid calling policy_fn during recompute #44661
SCh_zx创建于 5 天前
fix test_autograd to avoid calling policy_fn during recompute #44661
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 SCh_zx 的贡献）
atomgit-bot
5 天前 评论：
5 天前 评论：
变更摘要
该 PR 主要针对测试代码与禁用测试配置进行适配性调整：在 test/test_autograd.py 中将跳过 MPS 的装饰器从 skipIfMps 统一更名为 skipIfMPS，同时在 test/unsupported_test_cases/.pytorch-disabled-tests.json 中将两处测试用例名称中的设备标识由 privateuse1 改为 npu。改动仅涉及测试代码与配置，不涉及 autograd 核心逻辑。
主要改动
- 更新导入名称：将
test/test_autograd.py中从torch.testing._internal.common_utils导入的skipIfMps改为skipIfMPS。 - 统一装饰器命名：将
test/test_autograd.py内所有@skipIfMps装饰器批量替换为@skipIfMPS，覆盖test_parameter_resize、test_sparse_backward、test_where_functional等用例。 - 修正禁用测试用例名称：在
.pytorch-disabled-tests.json中将test_sparse_ctor_getter_backward_privateuse1_float64改为test_sparse_ctor_getter_backward_npu_float64，将test_view_copy_privateuse1改为test_view_copy_npu，使设备命名统一为npu。
ascend-robot
5 天前 评论：
5 天前 评论：
atomgit-bot
5 天前 评论：
5 天前 评论：
5 天前 添加了label：ascend-cla/yes
ascend-robot
5 天前 评论：
5 天前 评论：
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 | ✅ chenrayray, huangjingwei (2/2) | ✅ chenrayray (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
SCh_zx, thanks for your pull request. All authors of the commits have signed the CLA. 👍
ascend-robot
5 天前 评论：
5 天前 评论：
当前仓库存在以下 保护分支 ：
| Protected Branch | Version | Release |
|---|---|---|
| master | ||
| v2.7.1 | ||
| v2.12.0 | ||
| v2.11.0 | ||
| v2.10.0 | ||
| v2.9.0 | ||
| v2.7.1-26.1.0 | ||
| v2.11.0-26.1.0 | ||
| v2.12.0-26.1.0 | ||
| v2.10.0-26.1.0 | ||
| v2.9.0-26.1.0 | ||
| ci-test |
评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支（创建同步 PR）：
a) 如果当前 PR 是 Open 状态，同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged，将立即执行同步操作
注意：
- /sync 命令可以指定同步到多个分支，仅最后一个 /sync 命令生效
- 如果创建的同步 PR 不正确，可通过向同步 PR 的源分支提交轻量级 PR 完善，或使用 /close 命令关闭
5 天前 添加了label：ci-pipeline-running
ascend-robot
5 天前 评论：
5 天前 评论：
ascend docs pipeline is running...
5 天前 添加了label：docs-ci-pipeline-running
ascend-robot
5 天前 评论：
5 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
5 天前 删除了label：docs-ci-pipeline-running
5 天前 添加了label：docs-ci-pipeline-success
5 天前 删除了label：ci-pipeline-running
5 天前 添加了label：ci-pipeline-failed
ascend-robot
5 天前 评论：
5 天前 评论：
流水线 PR-pipeline_pytorch#61315 [ commitID：aa3db86f ] 运行失败
>>>代码风格自动修复执行成功（无修复内容）
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | codecheck_pre-commit | ✅ | >>> |
| check_error | ✅ | >>> | |
| lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | 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 : 停止流水线
此处折叠了10条事件消息 查看更多
5 天前 删除了label：ascend-cla/yes
ascend-robot
5 天前 评论：
5 天前 评论：
5 天前 添加了label：ascend-cla/yes
ascend-robot
5 天前 评论：
5 天前 评论：
ascend docs pipeline is running...
5 天前 删除了label：docs-ci-pipeline-success
ascend-robot
5 天前 评论：
5 天前 评论：
ascend docs pipeline is running...
5 天前 添加了label：docs-ci-pipeline-running
ascend-robot
5 天前 评论：
5 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
5 天前 删除了label：docs-ci-pipeline-running
5 天前 添加了label：docs-ci-pipeline-success
ascend-robot
5 天前 评论：
5 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
5 天前 添加了label：ci-pipeline-running
5 天前 删除了label：ci-pipeline-running
5 天前 添加了label：ci-pipeline-passed
ascend-robot
5 天前 评论：
5 天前 评论：
流水线 PR-pipeline_pytorch#61315 （重试第1次） [ commitID：aa3db86f ] 已完成
⚠️ 流水线执行结果已过期，当前 PR 最新 commit 为 04f88f9b，请重新运行流水线
>>>代码风格自动修复执行成功（无修复内容）
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_LibTorch_ARM | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| patch_test | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | codecheck_pre-commit | ✅ | >>> |
| check_error | ✅ | >>> | |
| lintrunner | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | 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 : 停止流水线
chenrayray
3 天前 评论：
3 天前 评论：
/approve
3 天前 添加了label：approved
Jingwei Huang
3 天前 评论：
3 天前 评论：
/lgtm
3 天前 添加了label：lgtm
3 天前 合入了pull request
ascend-robot
3 天前 评论：
3 天前 评论：
流水线 pytorch_gitcode_PR_multiVersion#14102 [ commitID：04f88f9b ] 已完成
【合入来源】
【修改方案】
原生社区中针对索引有所修改，修改用例以适配：
【资料变更】
【接口变更】
【功能验证】
【CheckList】