已合并
修复日志质量扫描发现不合理日志 #1297
liu-wei创建于 7 天前
修复日志质量扫描发现不合理日志 #1297
已合并
liu-wei创建于 7 天前
liu-wei
liu-wei成员
7 天前

Closes #750

描述

修复日志质量扫描发现的不合理日志,统一日志风格和代码格式。共 44 个文件,均为非功能性修改。

修复类型

类型 示例 文件数
日志文案规范化 "set tiling data error""Failed to set tiling data" 20+
日志换行修复 字符串中嵌入 \n 的多行日志 → 单行 3
代码格式对齐 clang-format 缩进/换行统一 15+
License 头标准化 HIT OpenBOAT 头 → 标准 CANN License 头 2
多余空行清理 连续 2 个空行 → 1 个 2

涉及文件(按模块)

模块 文件数
experimental/image 4
experimental/objdetect 1
image/ 各算子 tiling 20+
image/ aicpu kernel 4
objdetect/ 8
common/ 2
examples/ 1

功能影响

纯日志文案和代码格式修改,无任何逻辑变更。

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 liu-wei 的贡献)
liu-weiliu-wei成员
7 天前 创建了 pull request,commit a9f8ffb9
atomgit-bot
atomgit-bot
7 天前 评论:

变更摘要

本 PR 主要针对日志质量扫描发现的 53 条违规日志进行整改,覆盖各算子的 op_hostop_apiop_kernel_aicpu 及通用框架代码。改动以日志文案规范化为主:统一修正拼写错误、中文/全角标点、多余空格、字符串续行符与结尾换行符,并统一 tiling 数据初始化失败等日志的标准表达;同时伴随大量 clang-format 风格的代码排版调整,整体不改变算子核心逻辑。

主要改动

  • 统一 tiling 初始化失败日志文案: 将多处 "set tiling data error""memset tiling data error" 统一为 "Failed to set tiling data""Failed to memset tiling data",涉及 add_example_tiling.cpproi_align_v2_tiling.cppcrop_tiling_arch35.cppiou3d_tiling_arch35.cppanchor_response_flags_tiling.cpp 等众多 tiling 实现。
  • 修正拼写与标点错误: 将 faild 修正为 failed(如 aclnn_im2col_backward.cppupsample_nearest3d_grad_infershape.cpp),将 Compute failed 统一为 computation failed(如 non_max_suppression_v3_aicpu.cppscale_and_translate_aicpu.cpp),并将 及多余空格修正为标准英文标点。
  • 清理日志字符串格式: 合并多行字符串中的 \ 续行、移除日志末尾 \n(如 aclnn_grid_sampler2d.cppaclnn_grid_sampler3d.cppiou_v2.cpp),并优化措辞,如 "ub is not enough"/"ub not enough" 改为 "ub is insufficient""Format error." 改为 "Invalid format."
  • 代码格式统一调整: 对函数签名换行、IMPL_OP_OPTILING 链式调用、成员变量对齐及文件末尾换行进行 clang-format 风格整理(如 rgb2yuv422_tiling.cppyuv444_to_yuv422_tiling.cppgrid_sampler2_d_grad_tiling.cpp)。
  • 个别日志实参调整: aipp_tiling.cpp"aippConfigPath is: %s" 的格式化实参由 configData 改为字符串常量 "config_path"
likedislike
atomgit-bot
atomgit-bot
7 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
7 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
7 天前 评论:

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
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
experimental 於欣洁, 任如海, yue-ma, 王林木 (4/2) 王林木, 於欣洁 (2/1)
image/grid_sample 任如海, 王林木, yue-ma (3/2) 王林木 (1/1)
image/grid_sampler2_d_grad 王林木, 任如海, yue-ma (3/2) 王林木 (1/1)
image/resize_bicubic_v2 王林木, yue-ma (2/2) 王林木 (1/1)
image/resize_bilinear_v2 王林木, yue-ma (2/2) 王林木 (1/1)
image/resize_nearest_neighbor_v2 王林木, 任如海, yue-ma (3/2) 王林木 (1/1)
image/upsample_bicubic2d 任如海, yue-ma, 王林木 (3/2) 王林木 (1/1)
image/upsample_nearest3d_grad 王林木, 任如海, yue-ma (3/2) 王林木 (1/1)
image/upsample_trilinear3d_backward 王林木, 任如海, yue-ma (3/2) 王林木 (1/1)
objdetect/iou_v2 王林木, 於欣洁, yue-ma (3/2) 王林木, 於欣洁 (2/1)
objdetect/roi_align 王林木, 於欣洁, yue-ma (3/2) 王林木, 於欣洁 (2/1)
objdetect/roi_align_grad 王林木, 於欣洁, yue-ma (3/2) 王林木, 於欣洁 (2/1)
repo-cann/ops-cv 王林木, yue-ma (2/2) 王林木 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了46条消息 查看更多
yue-ma成员
6 天前 评论:

/lgtm

likedislike
llimwang成员
5 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:lgtmapproved
CANN-robotCANN-robot成员
5 天前 关闭了关联的issue
CANN-robotCANN-robot成员
5 天前 合入了pull request