[CI] Add ATen ops signature static check#79304
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #79304 +/- ##
===========================================
Coverage ? 100.00%
===========================================
Files ? 4
Lines ? 52
Branches ? 0
===========================================
Hits ? 52
Misses ? 0
Partials ? 0 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
CI报告基于以下代码生成(30分钟更新一次): 1 Required任务 : 41/48 通过
2 失败详情🔴 Coverage build — PR问题(置信度: 高)错误类型: PR问题 | 置信度: 高
关键日志:
修复建议:
关联变更: 🔴 Check approval — 需要 Approval(置信度: 高)错误类型: 需要 Approval | 置信度: 高
关键日志:
修复建议:
关联变更: 审批门禁,未指向具体源码文件。 🔴 Model-Benchmark / Benchmark test — 环境问题(置信度: 高)错误类型: 环境问题 | 置信度: 高
关键日志:
修复建议:
关联变更: 未发现与本 PR 变更文件直接相关。 🔴 CE-Framework / Test — 环境问题(置信度: 中)错误类型: 环境问题 | 置信度: 中
关键日志:
修复建议:
关联变更: 未发现与本 PR 变更文件直接相关。 |
5612d76 to
1efedac
Compare
1efedac to
eb9a0cd
Compare
PaddlePaddle-bot
left a comment
There was a problem hiding this comment.
🤖 Paddle-CI-Agent | pr_review |
2026-06-20 22:38:19
📋 Review 摘要
PR 概述:新增 ATen ops 兼容头文件签名静态检查,并对齐一批 compat ATen 入口签名。
变更范围:ci/static_check.sh、tools/check_aten_ops_signature.py、paddle/phi/api/include/compat/ATen/**、test/cpp/compat/**
影响面 Tag:[Execute Infrastructure] [User Experience]
问题
未发现新的阻塞性问题。历史问题状态见下,PR 规范问题在下面章节报,不在这里重复。
历史 Findings 修复情况
| Finding | 问题 | 状态 |
|---|---|---|
| F1 | 只收集新增 header,CI 未传 --all |
|
| F2 | transpose 维度参数在范围检查前收窄为 int |
✅ 已修复 |
📝 PR 规范检查
标题 Tag [CI] 不在当前模板枚举中,建议使用:
[Execute Infrastructure] Add ATen ops signature static check
PR 描述结构符合模板,是否引起精度变化 已填写为 否。
总体评价
本轮按风险优先审查了 CI 接入、checker 核心解析、transpose 收窄修复,以及索引/稀疏/record_stream 等 compat 入口变更;未全量覆盖其余机械签名调整。历史 F1 仍需要修复,否则新增静态检查在 CI 中仍无法覆盖本 PR 修改的既有 compat 头文件;历史 F2 已由当前 diff 修复。
PR Category
Execute Infrastructure
PR Types
Improvements
Description
本 PR 新增 ATen ops 兼容头文件函数签名静态检查,并修复现有 compat ATen ops 头文件中与 libtorch 2.9.1 public API 不一致的签名。
主要改动:
本地验证:
是否引起精度变化
否