Merged
ci: 自动发现增量测试并移除算子映射表 #48
wangliweiAdd on 19 days ago
ci: 自动发现增量测试并移除算子映射表 #48
Merged
wangliweiAdd on 19 days ago
wangliweiCollaborators
19 days ago

当前PR是否有AI参与:

[x] 否

[ ] 是
__1. AI Agent 平台:
__2. AI 模型:
__3. Prompt上下文 :

PR功能描述 / 为什么需要这个合入**:

当前增量测试依赖手工维护算子与测试的对应关系,新增算子时还需要修改 .gitcode/ 配置,容易遗漏。

本 PR 新增基于 Python AST 的测试选择器,通过源码和测试之间的直接、间接导入关系自动发现受影响用例,并接入现有 CI 执行脚本。新增算子和测试无需登记映射。

同时,移除 .gitcode/* 的默认全量触发规则,并修正选择器兜底逻辑:仅修改 CI 文件时跳过算子用例;同时修改源码或测试时正常增量选测。保留管理员在机器故障等场景调整执行逻辑的通道。

该PR关联的issue

(格式为fixes #<issue号>, 或者resolves #<issue号>): fixes #

希望检视人员了解:

改动类型 / Change Type

  • [❌ ] Bug 修复 / Bug Fix
  • [ ❌] 新功能 / New Feature
  • [❌ ] 性能优化 / Performance
  • [ ❌] 代码重构 / Refactoring
  • [❌ ] 文档更新 / Documentation
  • [ ✅] 测试相关 / Test
  • [ ❌] 其它 / Other

测试信息 / Testing

  • [✅ ] 单元测试通过 / UT passed
  • [ ✅] 集成测试通过 / ST passed
  • [ ✅] 人工验证通过 / Manual verified

检查清单 / Checklist

  • [✅ ] 代码符合规范 / Code follows style guide
  • [✅ ] 测试添加并通过 / Tests added and passed
  • [ ✅] 文档已更新 / Docs updated if needed
  • [ ✅] 无硬编码敏感信息 / No secrets hardcoded
  • [ ✅] 提交信息符合规范 / Commit message follows convention
likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for wangliwei's contribution.)
WwangliweiCollaborators
19 days ago Created pull request, commit 626ac976
CANN-robotCANN-robotCollaborators
19 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
19 days ago Comments:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

✅ Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
.gitcode/** ✅ tanghaoran7, ChaoyangJi, kevin365 (3/2) ✅ tanghaoran7 (1/1)
repo-cann/ops-tilelang ✅ kevin365, ChaoyangJi (2/2) ✅ kevin365, ChaoyangJi (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
CANN-robotCANN-robotCollaborators
19 days ago Set mywaaagh_admin,songchangxia,tanghaoran7,ChaoyangJi,kevin365,shentukangjun,monologue815,jiang-lirui,Allan_Yu,jiawei_zhuang,liudan12,Larry_liuyi,mabing1118 as approver
CANN-robotCANN-robotCollaborators
19 days ago Set mywaaagh_admin,songchangxia,tanghaoran7,ChaoyangJi,kevin365,shentukangjun,monologue815,jiang-lirui,Allan_Yu,jiawei_zhuang,liudan12,Larry_liuyi,mabing1118 as approver
50 messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
16 days ago Added label: lgtm
ChaoyangJi
ChaoyangJiCollaborators
16 days ago Comments:

/approve

likedislike
tanghaoran
tanghaoranCollaborators
15 days ago Comments:

/approve

likedislike
CANN-robotCANN-robotCollaborators
15 days ago Added label: approved
CANN-robotCANN-robotCollaborators
15 days ago Merged pull request