已合并
chore(precision-test): 同步 test_selector 上游并适配 torch_npu 改为 pytorch (#18) #44
tzing_t创建于 3 天前
chore(precision-test): 同步 test_selector 上游并适配 torch_npu 改为 pytorch (#18) #44
已合并
tzing_t创建于 3 天前
从precision合入到master
tzing_t成员
3 天前

变更说明

test_selector 上游近期调整了 pytorch 仓库的接入方式(repo 注册 key torch_npu→pytorch、PR 源 GitCode→GitHub),并新增首次绑定豁免规则。precision-test 内嵌的 test_selector 包需同步上游,否则 action 入参契约与 Python CLI choices 不一致会导致运行时报错。

变更内容

  • Python 包同步:src/ 与 dist/test_selector/ 的 8 个 py 文件 + README.md 同步至 test_selector 上游最新
    • 首次绑定豁免规则:纯插入块中绑定全新名字的惰性赋值不再记录锚点(diff_parser.py / change_detector.py)
    • repos 注册 key torch_npu→pytorch;TorchNpuAdapter 改用 --github-pr pytorch/pytorch#N,product_code_prefix 改为 torch/
  • 入参契约适配:
    • validators.js AVAILABLE_REPOS 替换 torch_npu 为 pytorch
    • action.yml repo-name/github-pr/gitcode-pr 描述更新(gitcode-pr 保留为通用入口,与 github-pr 互斥)
  • 文档适配:README.md torch_npu 示例改为 pytorch + github-pr,差异对照表/环境变量表/覆盖率布局文案同步
  • 重新打包:ncc 重新打包 dist/index.js,内嵌 AVAILABLE_REPOS 已含 pytorch

关联 Issue

  • Fixes #18
  • Refs #15(初始三仓统一重构)

测试计划

风险

  • repo-name 合法值由 torch_npu 变为 pytorch:使用旧值 torch_npu 的存量工作流需更新为 pytorch,否则 validators.js 会拒绝
  • torch_npu.py 文件名与 TorchNpuAdapter 类名未改(与上游一致,仅注册 key 和属性变化)

变更统计

21 files changed, 595 insertions(+), 180 deletions(-)

likedislike
Pull Request已成功合入, 合并人@openLiBingCI
(感谢 tzing_t 的贡献)
Ttzing_t成员
3 天前 创建了 pull request,commit 6b41750d
Ttzing_t成员
3 天前 关联了issue:【precision-test】重构为三仓统一 test_selector 插件,【precision-test】同步 test_selector 上游并适配 torch_npu 改为 pytorch
openLiBingCI成员
3 天前 评论:

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
repo-openlibing/test-action ✅ dd4574bb, nane, lsc941017 (3/2) ✅ dd4574bb (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
OopenLiBingCI成员
3 天前 添加了label:ci-pipeline-running
Ttzing_t成员
3 天前 添加了label:ai-assisted;删除了label:ci-pipeline-running
OopenLiBingCI成员
3 天前 添加了label:ci-pipeline-running
OopenLiBingCI成员
3 天前 添加了label:ci-pipeline-passed
OopenLiBingCI成员
3 天前 删除了label:ci-pipeline-running
openLiBingCI成员
3 天前 评论:
流水线 PR-pipeline_test-action#64 [ commitID:4711c4f8 ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build构建 ✅ COMPLETED >>>
病毒扫描 ✅ COMPLETED >>>
恶意代码检查 antipoison ✅ COMPLETED >>>
编码安全与规范检查 安全编码扫描 ✅ COMPLETED >>>
开源片段检查 开源漏洞和license扫描 ✅ COMPLETED >>>
流水线 PR-pipeline_test-action ✅ COMPLETED >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
nane成员
2 天前 评论:

/lgtm

likedislike
lishaocun成员
2 天前 评论:

/lgtm

likedislike
OopenLiBingCI成员
2 天前 添加了label:lgtm
dd4574bb成员
2 天前 评论:

/approve

likedislike
OopenLiBingCI成员
2 天前 添加了label:approved
openLiBingCI成员
2 天前 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: dd4574bb.
Reviewers who wrote a comment of /lgtm are: dd4574bb, nane, lsc941017.

likedislike
OopenLiBingCI成员
2 天前 合入了pull request,合并节点 SHA:8fbd4e6829516bc81d9fa923781450dfc1ec3766