已合并
cleanCode修改 #622
huangzhiyuan创建于 7月9日
cleanCode修改 #622
已合并
huangzhiyuan创建于 7月9日
huangzhiyuan
huangzhiyuan成员
7月9日

描述

cleanCode修改

关联的Issue

https://gitcode.com/cann/opbase/issues/264

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 huangzhiyuan 的贡献)
atomgit-bot
atomgit-bot
7月9日 评论:

变更摘要

此 PR 主要涉及两项改动的组合:一是在 classify_rule.yaml 中将 atvoss 相关路径拆分为更细粒度的 reduceutil 子目录,以调整代码分类规则;二是在 vec.h 中修复 Cast 构造函数中未使用参数 count 的编译器警告。

主要改动

  • classify_rule.yaml 路径细化:将 ops-base/pkg_inc/op_common/atvoss/ 拆分为 ops-base/pkg_inc/op_common/atvoss/reduceops-base/pkg_inc/op_common/atvoss/util,同时将 ops-base/src/op_common/atvoss/ 细化为 ops-base/src/op_common/atvoss/reduce,使代码风格分类规则更加精确。
  • vec.h 中未使用参数警告修复:在 Cast 结构体的 Cast(R& dst, T& scalar, int count) 构造函数中增加 (void) count;,以消除因 count 参数未被使用而触发的编译器警告。
likedislike
atomgit-bot
atomgit-bot
7月9日 评论:

代码审查


审查总结

文件 审查结论
pkg_inc/op_common/atvoss/util/vec.h ✅ 无问题 — (void) count; 是正确的未使用参数抑制写法
classify_rule.yaml ⚠️ 1 个 P3 低置信度发现 — VC2 路径收窄可能遗漏 broadcast/elewise 目录
  • P0–P2 发现数: 0
  • P3 发现数: 1(低置信度,需确认意图)
  • 整体风险评估: 低。vec.h 的改动是纯编译警告抑制,零风险。YAML 配置改动若为有意为之则同样无风险;若为疏忽,仅影响风格检查覆盖范围,不直接影响运行时正确性或安全性。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
CANN-robotCANN-robot成员
7月9日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7月9日 评论:

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
include 王永光, wangrui_, jia0liang (3/2) 王永光 (1/1)
pkg_inc jia0liang, wangrui_, 王永光 (3/2) 王永光 (1/1)
repo-cann/opbase jia0liang, wangrui_ (2/2) jia0liang (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
7月9日 将zhou-qilong,wangrui_,rxtfeng,loov1,jia0liang,songkai111,llimwang,gubaocheng设为评审人
此处折叠了49条消息 查看更多
CANN-robotCANN-robot成员
7月15日 添加了label:lgtm
wangyongguang成员
7月16日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
7月16日 添加了label:approved
CANN-robotCANN-robot成员
7月16日 关闭了关联的issue
CANN-robotCANN-robot成员
7月16日 合入了pull request