已合并
log_fix #8828
Alpha123_创建于 17 天前
log_fix #8828
已合并
Alpha123_创建于 17 天前
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Alpha123_ 的贡献)
Alpha123_Alpha123_成员
17 天前 创建了 pull request,commit 40e89df8
Alpha123_Alpha123_成员
17 天前 关联了issue:[Documentation|文档反馈]: 【matmul算子】【非量化】日志易用性质整改
atomgit-bot
atomgit-bot
17 天前 评论:

变更摘要

本 PR 主要针对 matmul 相关算子的日志、注释、头文件文档与标识符命名进行可读性和易用性整改,属于文档更新类改动。变更内容集中在:统一修正 OP_LOGE/OP_LOGW/OP_LOGI 等日志文案中的拼写与语法错误(如 shouleshouldcan notcannotnot supportdoes not supportonly supportonly supports),修正接口头文件中的 Doxygen 参数描述(如 workspace_sizeworkspaceSize内存起址内存地址exectorexecutor),并纠正变量、函数与宏的拼写错误(如 CheckBMMTilingDataIsVaildCheckBMMTilingDataIsValidswizzlCountswizzleCountTRNASTRANS)。同时修正了文件头 \file 注释、头文件保护宏以及部分中文注释和文档说明,未涉及算子核心计算逻辑变更。

主要改动

  • 统一修正日志与错误提示文案:批量修改 matmul 各算子中的 OP_LOGE/OP_LOGW/OP_LOGI 消息,纠正 shoulecan notnot supportreceive 等拼写/语法错误,并统一专有名词大小写(如 BatchMatMulWeightNzFusedMatmul),提升日志可读性。
  • 修正头文件接口文档注释:将 aclnn_*.h 中的 Doxygen 参数描述由 workspace_sizeexector内存起址那种申的的 等更正为 workspaceSizeexecutor内存地址哪种申请的,并修正 aclnnTransposeBatchMatmul 等算子名大小写及 aclnn_mv.h 的数据类型支持描述。
  • 纠正标识符、变量与宏命名:将 CheckBMMTilingDataIsVaildCheckBMMTilingDataIsValidswizzlCountswizzleCountInt42Int32PackedTensorInt4ToInt32PackedTensorunique_executoruniqueExecutorExectorExecutorContigousContiguousTRNASTRANSSetAtomicnoneSetAtomicNone/AscendC::SetAtomicNone 等拼写错误统一修正。
  • 修正文件头注释与 include guard 宏:将大量源文件头 \file 注释中的 .cc 更正为 .cpp 并修正错误文件名,同时修正多个头文件结尾的保护宏(如 OP_API_INC_ADD_H_OP_API_INC_ADDMV_H_MMV3_MATMUL_KERNEL_H→对应文件的正确宏名)。
  • 清理冗余日志并修正中文注释:删除 RotateQuantAptTiling::PrintTilingData() 中冗余的 tailML1 日志,修正中文注释中的错别字与表述(如 判决门限判断门限申明声明内涵包含stroageShapestorageShape),并调整部分被注释掉的说明块格式。
likedislike
atomgit-bot
atomgit-bot
17 天前 评论:

代码审查

✅ 未发现问题

likedislike
CANN-robotCANN-robot成员
17 天前 添加了label:cann-cla/yes
此处折叠了172条消息 查看更多
范其瑞
范其瑞成员
8 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
8 天前 添加了label:lgtm
CANN-robotCANN-robot成员
8 天前 关闭了关联的issue
CANN-robotCANN-robot成员
8 天前 合入了pull request
CANN-robot
CANN-robot成员
8 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike