已合并
fix: 从 ops-cv 迁出 NormalizeBBox 原型 #1456
raoliang_sac创建于 9月2日
fix: 从 ops-cv 迁出 NormalizeBBox 原型 #1456
已合并
Pull Request已成功合入, 合并人@CANN-robot
(感谢 raoliang_sac 的贡献)9月2日 创建了 pull request,commit 9dafbb25
atomgit-bot
9月2日 评论:
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 组合发布。


不准确?
atomgit-bot
9月2日 评论:
9月2日 评论:
9月2日 添加了label:cann-cla/yes
CANN-robot
9月2日 评论:
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
/approveor/lgtm- Commenting
/approveimplies 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. 👍


CANN-robot
9月2日 评论:
9月2日 评论:
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设为评审人
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设为评审人
9月2日 将TianXL,jiangtao_rts,gitcode-chenjiao,hanxiaolong,gcw_kUomxQ2l,Andy-lb设为审查人
raoliang_sac
9月2日 评论:
9月2日 评论:
compile


9月2日 预合并成功(commit_id: 46e72a5ba48577262e3ea73032094f008f243adf)
CANN-robot
9月2日 评论:
9月2日 评论:
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |


9月2日 添加了label:ci-pipeline-running
此处折叠了6条事件消息 查看更多
9月2日 添加了label:cann-cla/yes
raoliang_sac
9月2日 评论:
9月2日 评论:
compile


9月2日 预合并成功(commit_id: af075b51052a54c9813042495416f53c24795d96)
CANN-robot
9月2日 评论:
9月2日 评论:
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |


9月2日 删除了label:ci-pipeline-failed
此处折叠了8条事件消息 查看更多
9月3日 删除了label:ci-pipeline-passed
CANN-robot
9月3日 评论:
9月3日 评论:
9月3日 添加了label:cann-cla/yes
9月3日 修改了pull request 的描述
raoliang_sac
9月3日 评论:
9月3日 评论:
compile


9月3日 预合并成功(commit_id: 44c56cefa32923198428c9e32c32a4ba469f4fc5)
CANN-robot
9月3日 评论:
9月3日 评论:
| 🚀 CI 流水线已启动 |
|---|
| 📋 执行详情: 点击查看流水线 |


9月3日 添加了label:ci-pipeline-running
9月3日 删除了label:ci-pipeline-running
9月3日 添加了label:ci-pipeline-passed
9月3日 添加了label:approved
9月3日 添加了label:lgtm
9月3日 合入了pull request
描述
从 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
无,商分同步主线原型归属。
测试
2275f5b6f44020f44d4f89c6caf83e3e90c62925,最终仅修改op_cv_proto_extend.h,未修改.clang-format。scripts/util/merge_proto.py对比基线与修改后结果:REG_OP 总数由 73 变为 72,唯一减少项为 NormalizeBBox,修改后无重复定义。ops_proto_cv.h执行g++ -std=c++17 -fsyntax-only,检查通过。git diff --check通过。文档更新
不涉及。
类型标签