master

cann-robotfix: 修复比较类算子示例资源泄漏、文档错误与代码格式问题

116bf445Add on 26 days agoHistory

Files	Last commit	Last update
aclnn_logical_xor.cpp	feat: 算子910B~910E区间判断追加IsRegBase()并统一DAV_3510硬编码判断以兼容后续Regbase芯片 Co-authored-by: hahaha22<wangrunze20@h-partners.com> # message auto-generated for no-merge-commit merge: !5467 merge q_aclnn_version into master feat: 算子910B~910E区间判断追加IsRegBase()并统一DAV_3510硬编码判断以兼容后续Regbase芯片 Created-by: hahaha22 Commit-by: hahaha22 Merged-by: cann-robot Description: ## 描述 以 RegBase 架构（NpuArch = DAV_3510，含 Ascend950 及后续 Regbase 芯片）兼容为目标，统一两类芯片判断写法： 1. **区间判断追加**：对 13 个算子 op_api 中 GetSocVersion() >= ASCEND910B && GetSocVersion() <= ASCEND910E 区间判断追加 || IsRegBase()，使 RegBase 平台在 SocVersion 区间之外时也复用 910B 支持分支（放开 BF16 dtype 校验） 2. **硬编码统一**：将 11 个算子文件中 case NpuArch::DAV_3510: 硬编码判断改为 IsRegBase() 谓词，后续 Regbase 芯片（IsRegBase 集合扩展新 arch 值时）无需逐文件修改即可自动命中 ## 修改内容一：区间判断追加（13 个文件，15 处） | 算子 | 文件 | 修改位置 | |------|------|----------| | less | math/less/op_api/aclnn_lt_scalar.cpp | GetDtypeSupportList、GetOutDtypeSupportList（2 处） | | less | math/less/op_api/aclnn_lt_tensor.cpp | GetDtypeSupportList、GetOutDtypeSupportList（2 处） | | less_equal | math/less_equal/op_api/aclnn_le_scalar.cpp | GetOutputDtypeSupportList（1 处） | | logical_or | math/logical_or/op_api/aclnn_logical_or.cpp | CheckSocVersionIsSupportBf16（1 处） | | maximum | math/maximum/op_api/aclnn_maximum.cpp | GetDtypeSupportList（1 处） | | minimum | math/minimum/op_api/aclnn_minimum.cpp | GetDtypeSupportList（1 处） | | mul | math/mul/op_api/aclnn_mul.cpp | GetDtypeSupportList（1 处） | | not_equal | math/not_equal/op_api/aclnn_logical_xor.cpp | CheckSocVersionIsSupportBf16（1 处） | | pow | math/pow/op_api/aclnn_pow.cpp | CheckSocVersionIsSupportBf16（1 处） | | pow | math/pow/op_api/aclnn_pow_tensor_tensor.cpp | CheckSocVersionIsSupportBf16（1 处） | | signbit | math/signbit/op_api/aclnn_signbit.cpp | GetDtypeSupportList（1 处） | | sub | math/sub/op_api/sub.cpp | IsAiCoreSupport（1 处） | | equal | math/tensor_equal/op_api/aclnn_equal.cpp | CheckSocVersionGe910B（1 处） | 其中 sub.cpp、aclnn_pow_tensor_tensor.cpp、aclnn_logical_xor.cpp、aclnn_equal.cpp 补充 #include "op_api/aclnn_check.h"。 pow.cpp 的 IsAiCoreSupport、aclnn_ne_scalar.cpp、aclnn_ne_tensor.cpp：区间判断前已有 if (IsRegBase()) 提前返回分支（追加会造成死条件），经评审不修改。 ## 修改内容二：硬编码判断统一（11 个文件，13 处） ### A类：switch 双 case（DAV_3510 与 DAV_2201 同分支，8处） switch 前置 if (IsRegBase(curArch)) 返回原 DAV_3510 分支列表，删除 case NpuArch::DAV_3510: fall-through 行： - Add：math/add/op_api/add.cpp、aclnn_add.cpp - Sub：math/sub/op_api/aclnn_sub.cpp、aclnn_rsub.cpp - div：math/div/op_api/div.cpp（补 include） - floordiv：math/floor_div/op_api/floordiv.cpp（补 include） - tanhgrad：math/tanh_grad/op_api/aclnn_tanh_backward.cpp - floormod：math/floor_mod/op_api/aclnn_remainder.cpp ### B类：switch 独立 case（DAV_3510 单独返回 REGBASE 专用列表，3处） switch 前置 if (IsRegBase(curArch)) 返回 REGBASE 列表，删除独立 case 块： - mul：math/mul/op_api/mul.cpp（REGBASE_AICORE_DTYPE_SUPPORT_LIST） - equal：math/equal/op_api/aclnn_eq_tensor.cpp、aclnn_eq_scalar.cpp（REGBASE_DTYPE_SUPPORT_LIST） ## 关联的Issue - #3100 （math仓910B~910E区间判断的RegBase兼容遗留清理：isclose恒真条件bug、8个同类判断文件未覆盖IsRegBase） ## 测试 - pre-commit 检查（clang-format/codespell/OAT 等）全部通过（含对修改文件的全量格式化） - 全部修改文件 g++ -fsyntax-only 语法验证通过 - 已触发 compile 编译 - 建议测试项： 1. RegBase（DAV_3510）平台验证上述算子 BF16 输入/输出 dtype 校验与 910B 平台一致 2. Ascend950 平台回归 A/B 类算子 dtype 选择行为不变 3. 回归 910B~910E、910/1980、310/310P 平台行为不受影响 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： ### 关联issue - [cann/ops-math#5512（报错 dtype 支持列表双层中括号修复，同改 aclnn_remainder.cpp/eq 系列）](https://gitcode.com/cann/ops-math/pull/5512) - [cann/ops-math#5519（RemainderScalarTensor Regbase 计算类型校验，同改 aclnn_remainder.cpp）](https://gitcode.com/cann/ops-math/pull/5519) - [cann/ops-nn#10101（ops-nn 侧同类 Regbase 判断统一）](https://gitcode.com/cann/ops-nn/pull/10101) See merge request: cann/ops-math!5467	2 months ago
aclnn_logical_xor.h	refactor: 规范化 math仓部分算子代码格式 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6203 merge ccc_1010 into master refactor: 规范化 math仓部分算子代码格式 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本次改动对 math/ 目录下 9 个算子（Equal、FloorDiv、Greater、GreaterEqual、Less、LessEqual、Maximum、Minimum、NotEqual）进行代码格式规范化，不涉及任何功能变更。 ### 改动原因 统一代码风格，符合 clang-format 规范，提升可读性与可维护性。 ### 改动方法 1. 按 clang-format 规范统一缩进、换行与对齐； 2. 补充缺失的文件末尾换行符； 3. 清理尾随空格与多余空行； 4. 修复配置文件注释拼写错误（build-in → built-in）； 5. 统一 Python 测试脚本 docstring 风格（''' → """）。 ## 关联的Issue - #3467 ## 测试 本次改动仅涉及代码格式调整，无功能变更，不涉及二级冒烟、算子泛化等验证。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：代码格式规范化（重构） See merge request: cann/ops-math!6203	30 days ago
aclnn_ne_scalar.cpp	refactor: 规范化 math仓部分算子代码格式 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6203 merge ccc_1010 into master refactor: 规范化 math仓部分算子代码格式 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本次改动对 math/ 目录下 9 个算子（Equal、FloorDiv、Greater、GreaterEqual、Less、LessEqual、Maximum、Minimum、NotEqual）进行代码格式规范化，不涉及任何功能变更。 ### 改动原因 统一代码风格，符合 clang-format 规范，提升可读性与可维护性。 ### 改动方法 1. 按 clang-format 规范统一缩进、换行与对齐； 2. 补充缺失的文件末尾换行符； 3. 清理尾随空格与多余空行； 4. 修复配置文件注释拼写错误（build-in → built-in）； 5. 统一 Python 测试脚本 docstring 风格（''' → """）。 ## 关联的Issue - #3467 ## 测试 本次改动仅涉及代码格式调整，无功能变更，不涉及二级冒烟、算子泛化等验证。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：代码格式规范化（重构） See merge request: cann/ops-math!6203	30 days ago
aclnn_ne_scalar.h	fix: 修复比较类算子示例资源泄漏、文档错误与代码格式问题 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6328 merge code_check_1013 into master fix: 修复比较类算子示例资源泄漏、文档错误与代码格式问题 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本 PR 针对 equal、greater、greater_equal、less、less_equal、maximum、minimum、not_equal 共 8 个比较类算子，修复代码检视（code check）发现的问题，具体包括： 1. 修复示例代码资源管理缺陷：examples/*.cpp 改用 RAII（std::unique_ptr + 自定义删除器）统一管理 aclrtStream、device 侧内存、aclTensor、aclScalar 等资源，避免异常路径下资源泄漏。 2. 修复头文件复制粘贴错误：maximum_struct.h 中 tiling key 误写为 Minimum，修正为 Maximum；greater_proto.h 中精度约束公式 (x2 - x1) 修正为 (x1 - x2)。 3. 修复文档注释错误：aclnn_eq_tensor.h 中 workspaceSize 描述引用错误接口名 aclnnInplaceAddGetWorkspaceSize，修正为 aclnnInplaceEqTensorGetWorkspaceSize。 4. 修复文档描述问题：修正 README 与 docs 中失效链接、参数名（self → tensors）、错误码行数、数据类型表格填充、章节标题等。 5. 修复测试用例数据：less 算子 ST 用例 ttk_kernel_less_st.csv 中 uint64 infnan 用例数据类型由 uint64 修正为 float32。 6. 规范代码格式：清理行尾空格、规范函数签名换行、补齐文件结尾换行符等。 ## 关联的Issue - #3619 ## 测试 本次变更主要为示例代码、头文件注释、文档与 ST 测试数据的修正，不涉及算子 kernel 计算逻辑变更。 ## 文档更新 更新了 equal/greater/greater_equal/less/less_equal/not_equal 的 README.md，以及 equal/less/maximum/minimum/not_equal 的 docs/*.md。 ## 类型标签 - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6328	26 days ago
aclnn_ne_tensor.cpp	fix: 修复 equal/not_equal/less/less_equal 算子日志拼写/语义/copy-paste 错误 Co-authored-by: sun-yibo1<sunyibo1@hisilicon.com> # message auto-generated for no-merge-commit merge: !5187 merge fix/comparison-log-audit into master fix: 修复 equal/not_equal/less/less_equal 算子日志拼写/语义/copy-paste 错误 Created-by: sun-yibo1 Commit-by: sun-yibo1 Merged-by: cann-robot Description: ## 描述 参照 log_audit_report.md，修复 ops-math 仓库比较类算子（equal/not_equal/less/less_equal）的日志拼写、语义及 copy-paste 错误。本 PR 由原 PR #5158 拆分而来（拆分方案：算术/比较/极值/其他 共 4 个 PR），仅修改日志文案/注释，不改动 REG_OP/INPUT/OUTPUT/ATTR 等代码定义。 ### 改动原因 log_audit_report.md 中 ops-math 仓库共发现 107 个日志问题，此前已通过 PR #4779（floor_mod/floor_div）、PR #4949（mod/mul/pow）、PR #5117（bitwise_and）修复了部分。原 PR #5158 一次性提交剩余 33 个文件的修复，为便于评审与独立合入，按算子类别拆分为 4 个 PR。本 PR 为拆分后的第 2 个，覆盖比较类（6 文件）。 ### 改动方法 仅修改日志文案/注释中的拼写、语义、copy-paste 错误，同时按 pre-commit clang-format 要求对文件做格式化。 ### 涉及修复（6 文件） **拼写/语法：** - is not support → is not supported（not_equal） - only support → only supports（equal） **copy-paste/语义：** - less：tag LessTiling → TilingForLess - less_equal：InferShape4Less → InferShape4LessEqual；tag LessEqualTiling → Tiling4LessEqual ## 关联的Issue - #2950 ## 测试 - 仅日志文案/格式修改，无逻辑变更，不影响编译与功能。 - pre-commit 全部代码 hook 通过（trailing-whitespace / end-of-file-fixer / clang-format / codespell / check-* / detect-private-key）。 ## 文档更新 无。 ## 类型标签 - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： ## AI/Agent生成声明 - [x] AI辅助编写 See merge request: cann/ops-math!5187	2 months ago
aclnn_ne_tensor.h	fix: 修复比较类算子示例资源泄漏、文档错误与代码格式问题 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6328 merge code_check_1013 into master fix: 修复比较类算子示例资源泄漏、文档错误与代码格式问题 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本 PR 针对 equal、greater、greater_equal、less、less_equal、maximum、minimum、not_equal 共 8 个比较类算子，修复代码检视（code check）发现的问题，具体包括： 1. 修复示例代码资源管理缺陷：examples/*.cpp 改用 RAII（std::unique_ptr + 自定义删除器）统一管理 aclrtStream、device 侧内存、aclTensor、aclScalar 等资源，避免异常路径下资源泄漏。 2. 修复头文件复制粘贴错误：maximum_struct.h 中 tiling key 误写为 Minimum，修正为 Maximum；greater_proto.h 中精度约束公式 (x2 - x1) 修正为 (x1 - x2)。 3. 修复文档注释错误：aclnn_eq_tensor.h 中 workspaceSize 描述引用错误接口名 aclnnInplaceAddGetWorkspaceSize，修正为 aclnnInplaceEqTensorGetWorkspaceSize。 4. 修复文档描述问题：修正 README 与 docs 中失效链接、参数名（self → tensors）、错误码行数、数据类型表格填充、章节标题等。 5. 修复测试用例数据：less 算子 ST 用例 ttk_kernel_less_st.csv 中 uint64 infnan 用例数据类型由 uint64 修正为 float32。 6. 规范代码格式：清理行尾空格、规范函数签名换行、补齐文件结尾换行符等。 ## 关联的Issue - #3619 ## 测试 本次变更主要为示例代码、头文件注释、文档与 ST 测试数据的修正，不涉及算子 kernel 计算逻辑变更。 ## 文档更新 更新了 equal/greater/greater_equal/less/less_equal/not_equal 的 README.md，以及 equal/less/maximum/minimum/not_equal 的 docs/*.md。 ## 类型标签 - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6328	26 days ago
not_equal.cpp	refactor: 规范化 math仓部分算子代码格式 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6203 merge ccc_1010 into master refactor: 规范化 math仓部分算子代码格式 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本次改动对 math/ 目录下 9 个算子（Equal、FloorDiv、Greater、GreaterEqual、Less、LessEqual、Maximum、Minimum、NotEqual）进行代码格式规范化，不涉及任何功能变更。 ### 改动原因 统一代码风格，符合 clang-format 规范，提升可读性与可维护性。 ### 改动方法 1. 按 clang-format 规范统一缩进、换行与对齐； 2. 补充缺失的文件末尾换行符； 3. 清理尾随空格与多余空行； 4. 修复配置文件注释拼写错误（build-in → built-in）； 5. 统一 Python 测试脚本 docstring 风格（''' → """）。 ## 关联的Issue - #3467 ## 测试 本次改动仅涉及代码格式调整，无功能变更，不涉及二级冒烟、算子泛化等验证。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：代码格式规范化（重构） See merge request: cann/ops-math!6203	30 days ago
not_equal.h	refactor: 规范化 math仓部分算子代码格式 Co-authored-by: cai-chengchao<caichengchao1@h-partners.com> # message auto-generated for no-merge-commit merge: !6203 merge ccc_1010 into master refactor: 规范化 math仓部分算子代码格式 Created-by: cai-chengchao Commit-by: cai-chengchao Merged-by: cann-robot Description: ## 描述 本次改动对 math/ 目录下 9 个算子（Equal、FloorDiv、Greater、GreaterEqual、Less、LessEqual、Maximum、Minimum、NotEqual）进行代码格式规范化，不涉及任何功能变更。 ### 改动原因 统一代码风格，符合 clang-format 规范，提升可读性与可维护性。 ### 改动方法 1. 按 clang-format 规范统一缩进、换行与对齐； 2. 补充缺失的文件末尾换行符； 3. 清理尾随空格与多余空行； 4. 修复配置文件注释拼写错误（build-in → built-in）； 5. 统一 Python 测试脚本 docstring 风格（''' → """）。 ## 关联的Issue - #3467 ## 测试 本次改动仅涉及代码格式调整，无功能变更，不涉及二级冒烟、算子泛化等验证。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：代码格式规范化（重构） See merge request: cann/ops-math!6203	30 days ago