[Execute Infrastructure] Align ATen compat checks with PyTorch 2.13.0#79457
[Execute Infrastructure] Align ATen compat checks with PyTorch 2.13.0#79457youge325 wants to merge 1 commit into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates Paddle’s ATen C++ compat static signature-check baseline from PyTorch 2.12.1 to 2.13.0, removing compat stubs/tests for APIs that are no longer part of PyTorch 2.13.0’s public C++ surface (notably TensorBase::has_names and Tensor::rename).
Changes:
- Bump the static-check/install hint Torch version from
2.12.1to2.13.0in both CI and the Python checker. - Remove compat header stub
ATen/ops/rename.hand theTensor::rename/TensorBase::has_namescompat declarations. - Remove the dedicated C++ compat unit test and CMake registration that only exercised the removed APIs.
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tools/check_aten_ops_signature.py | Updates Torch install hint used by the signature checker to PyTorch 2.13.0. |
| ci/static_check.sh | Updates CI static-check to install torch==2.13.0 for signature scanning. |
| paddle/phi/api/include/compat/ATen/core/TensorBase.h | Removes TensorBase::has_names compat API. |
| paddle/phi/api/include/compat/ATen/core/TensorBody.h | Removes Tensor::rename compat API declaration. |
| paddle/phi/api/include/compat/ATen/Functions.h | Drops include of the removed ATen/ops/rename.h. |
| paddle/phi/api/include/compat/ATen/ops/rename.h | Deletes the compat stub header for Tensor::rename. |
| test/cpp/compat/ATen_basic_test.cc | Removes unit coverage that referenced has_names(). |
| test/cpp/compat/CMakeLists.txt | Removes the ATen_rename_test target registration. |
| test/cpp/compat/ATen_rename_test.cc | Deletes the compat unit tests for Tensor::rename. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| pip install --target "${aten_ops_signature_torch_target}" \ | ||
| torch==2.12.1 --index-url https://download.pytorch.org/whl/cpu 1>nul | ||
| torch==2.13.0 --index-url https://download.pytorch.org/whl/cpu 1>nul |
risemeup1111
left a comment
There was a problem hiding this comment.
已完成首轮评审。整体变更集中在将 ATen compat 静态签名基线对齐到 PyTorch 2.13.0,并移除不再公开的 rename / has_names 兼容接口;未发现需要阻塞合入的正确性问题。
有一处低优先级脚本清理建议已放在行级评论里,作者回复或顺手修正后即可。
| aten_ops_signature_torch_target=$(mktemp -d) | ||
| pip install --target "${aten_ops_signature_torch_target}" \ | ||
| torch==2.12.1 --index-url https://download.pytorch.org/whl/cpu 1>nul | ||
| torch==2.13.0 --index-url https://download.pytorch.org/whl/cpu 1>nul |
There was a problem hiding this comment.
1>nul 在 Linux bash 下会把 stdout 重定向到当前目录的字面文件 nul,不会丢弃输出;这条 static check 路径触发后可能在 workspace 留下多余文件,并且和脚本里其他 /dev/null 用法不一致。请改成 /dev/null。
处理要求:请针对该评论进行回复(同意并已修改请回复 Done,不同意请说明理由)。
| torch==2.13.0 --index-url https://download.pytorch.org/whl/cpu 1>nul | |
| torch==2.13.0 --index-url https://download.pytorch.org/whl/cpu 1>/dev/null |
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-07-10 21:57:20
📋 Review 摘要
PR 概述:将 ATen C++ compat 静态签名基线从 PyTorch 2.12.1 升级到 2.13.0,并移除 2.13.0 不再公开的 rename/has_names 兼容接口。
变更范围:ci/static_check.sh、tools/check_aten_ops_signature.py、paddle/phi/api/include/compat/ATen/、test/cpp/compat/
影响面 Tag:[Execute Infrastructure]
问题
未发现阻塞性问题。PR 规范问题在下面章节报,不在这里重复。
📝 PR 规范检查
符合规范。标题使用 [Execute Infrastructure],PR 描述包含 PR Category、PR Types、Description、是否引起精度变化 四个必填 section,精度变化填写为 否。
总体评价
本轮按风险优先审查了 ATen compat 静态检查基线、删除的 compat public API 及对应测试注册。rename / has_names 的残留引用已清除,签名检查脚本和本地安装提示同步到了同一 torch 版本,未发现需要阻塞合入的问题。
CI报告基于以下代码生成(30分钟更新一次): 1 Required任务 : 39/48 通过
2 失败详情🔴 DeepMD-Kit-Test / DeepMD-Kit Integration Test — 环境问题(置信度: 高)失败用例: 不适用(安装 PaddlePaddle 阶段失败)
关键日志:
修复建议:
关联变更: 未发现与 PR 代码变更直接相关。 🔴 Static-Check / Test — PR问题(置信度: 高)失败用例: 不适用(Static check 阶段失败)
关键日志:
修复建议:
关联变更: 🔴 Coverage build — 环境问题(置信度: 高)失败用例: 不适用(Coverage build 阶段失败)
关键日志:
修复建议:
关联变更: 未发现与 PR 代码变更直接相关。 🔴 Fleet Unit test (multi-card) — 环境问题(置信度: 高)失败用例: 不适用(容器启动阶段失败)
关键日志:
修复建议:
关联变更: 未发现与 PR 代码变更直接相关。 🔴 Fleet Unit test (single card) — 环境问题(置信度: 高)失败用例: 不适用(容器启动阶段失败)
关键日志:
修复建议:
关联变更: 未发现与 PR 代码变更直接相关。 🔴 Check approval — 需要 Approval(置信度: 高)
修复建议:
🔴 CE-Framework / Infer — 环境问题(置信度: 中)失败用例: 不适用(PaddleTest 下载/解压阶段失败)
关键日志:
修复建议:
关联变更: 未发现与 PR 代码变更直接相关。 🔴 Mac-CPU / Build and test — 未知(置信度: 低)失败用例: 未能提取
关键日志:
修复建议:
关联变更: 未分析。 |
|
你的PR提交成功,感谢你对开源项目的贡献! |
PR Category
Execute Infrastructure
PR Types
BC Breaking
Description
将 ATen C++ compat 静态签名基线从 PyTorch 2.12.1 升级到 2.13.0。
torch==2.13.0CPU wheel,并同步更新本地安装提示。TensorBase::has_names、Tensor::rename和ATen/ops/rename.h。--all扫描通过。该变更会移除不再属于 PyTorch 2.13.0 public C++ API 的 compat 接口,因此标记为 BC Breaking。
是否引起精度变化
否