已合并
fix: 从 ops-cv 迁出 NormalizeBBox 原型 #1456
fix: 从 ops-cv 迁出 NormalizeBBox 原型 #1456
已合并
raoliang_sac创建于 9月2日
raoliang_sac
9月2日

描述

从 ops-cv 的 9.1.0 商分删除 NormalizeBBox 的完整 REG_OP 定义,由 ops-nn 商分统一维护。

该原型已等价迁入 ops-nn/common/inc/op_graph/op_nn_proto_extend.h,输入、输出、数据类型、reversed_box 属性及默认值保持不变,并增加防重定义保护。

配套 PR:

本 PR 需要与 ops-nn 配套变更组合入,避免单边发布造成原型缺失。

关联的Issue

无,商分同步主线原型归属。

测试

  • 已 rebase 至远端最新 9.1.0 基线 2275f5b6f44020f44d4f89c6caf83e3e90c62925,最终仅修改 op_cv_proto_extend.h,未修改 .clang-format。
  • 执行真实 pre-commit 增量检查,clang-format 与 OAT 均通过。
  • 使用仓库真实 scripts/util/merge_proto.py 对比基线与修改后结果:REG_OP 总数由 73 变为 72,唯一减少项为 NormalizeBBox,修改后无重复定义。
  • 使用指定 CANN 头文件对生成的 ops_proto_cv.h 执行 g++ -std=c++17 -fsyntax-only,检查通过。
  • 跨仓核对 NormalizeBBox 定义数量:ops-cv 为 0,配套 ops-nn 为 1,组合后为 1。
  • git diff --check 通过。

文档更新

不涉及。

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 raoliang_sac 的贡献)
Rraoliang_sac
9月2日 创建了 pull request,commit 9dafbb25
atomgit-bot
atomgit-bot
9月2日 评论:

变更摘要

本 PR 将 NormalizeBBox 算子原型从 ops-cv 商分迁出:删除 common/inc/op_graph/op_cv_proto_extend.h 中 NormalizeBBox 的完整 REG_OP 定义(共 21 行),改由 ops-nn 商分统一维护(等价原型已迁入 ops-nn/common/inc/op_graph/op_nn_proto_extend.h,并增加防重定义保护)。该改动主要聚焦原型归属的同步,不涉及接口或行为变化,且需与 ops-nn 配套变更组合发布,避免单边发布造成原型缺失。

主要改动

  • 删除 NormalizeBBox 原型定义:从 common/inc/op_graph/op_cv_proto_extend.h 中移除 REG_OP(NormalizeBBox) 至 .OP_END_FACTORY_REG(NormalizeBBox) 的完整定义块,对应删除 21 行。
  • 保持原型语义不变:被删除定义中的输入 boxes(DT_FLOAT16/DT_FLOAT)、shape_hw(DT_INT32)、输出 y 及属性 reversed_box(Bool,默认 false)与 ops-nn 侧迁入的定义完全一致,不改变算子接口。
  • 迁移至 ops-nn 统一维护:NormalizeBBox 原型改由 ops-nn 的 op_nn_proto_extend.h 维护,组合生成后仅保留一份,避免双商分重复定义;需与 ops-nn 配套 MR 组合发布。
likedislike
不准确?
atomgit-bot
atomgit-bot
9月2日 评论:

代码审查

✅ 未发现问题

likedislike
不准确?
CANN-robotCANN-robot成员
9月2日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
9月2日 评论:

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
9.1.0 ✅ Andy-lb, yourealize (2/2) ✅ Andy-lb (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
9月2日 评论:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
9月2日 将yue-ma,zhou-qilong,rxtfeng,loov1,tang-lei01,zhangzijie,TianXL,yourealize,zl_hw,liu-wei,jiangtao_rts,gitcode-chenjiao,hanxiaolong,pingchuantang,llimwang,gubaocheng,wangyongguang,gcw_kUomxQ2l,Andy-lb设为评审人
CANN-robotCANN-robot成员
9月2日 将TianXL,jiangtao_rts,gitcode-chenjiao,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
raoliang_sac
9月2日 评论:

compile

likedislike
Rraoliang_sac
9月2日 预合并成功(commit_id: 46e72a5ba48577262e3ea73032094f008f243adf)
CANN-robot
CANN-robot成员
9月2日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月2日 添加了label:ci-pipeline-running
此处折叠了6条事件消息 查看更多
CANN-robotCANN-robot成员
9月2日 添加了label:cann-cla/yes
raoliang_sac
9月2日 评论:

compile

likedislike
Rraoliang_sac
9月2日 预合并成功(commit_id: af075b51052a54c9813042495416f53c24795d96)
CANN-robot
CANN-robot成员
9月2日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月2日 删除了label:ci-pipeline-failed
此处折叠了8条事件消息 查看更多
CANN-robotCANN-robot成员
9月3日 删除了label:ci-pipeline-passed
CANN-robot
CANN-robot成员
9月3日 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
CANN-robotCANN-robot成员
9月3日 添加了label:cann-cla/yes
Rraoliang_sac
9月3日 修改了pull request 的描述
raoliang_sac
9月3日 评论:

compile

likedislike
Rraoliang_sac
9月3日 预合并成功(commit_id: 44c56cefa32923198428c9e32c32a4ba469f4fc5)
CANN-robot
CANN-robot成员
9月3日 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
9月3日 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月3日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
9月3日 添加了label:ci-pipeline-passed
Andy-lb成员
9月3日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
9月3日 添加了label:approved
陈展熹成员
9月3日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
9月3日 添加了label:lgtm
CANN-robotCANN-robot成员
9月3日 合入了pull request