master

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

116bf445Add on 22 days agoHistory

Files	Last commit	Last update
aclnn_maximum.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	1 month ago
aclnn_maximum.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	22 days ago
aclnn_maxn.cpp	refactor: 继续迁移非 experimental 公共头文件引用 Co-authored-by: clinglai0517<laichangling@huawei.com> # message auto-generated for no-merge-commit merge: !6220 merge refactor/math-prefix-common-headers-1008 into master refactor: 继续迁移非 experimental 公共头文件引用 Created-by: clinglai0517 Commit-by: clinglai0517 Merged-by: cann-robot Description: ## 描述 继续将 ops-math 的公共 op_api 头文件引用迁移到带 _math 后缀的名称，减少多个算子仓库集成时同名头文件误引用的风险。 本次更新 35 个非 experimental 文件中的 35 处引用： - level2_base.h → level2_base_math.h：11 处。 - level2_base_caculation.h → level2_base_caculation_math.h：1 处。 - op_api_def.h → op_api_def_math.h：23 处。 对涉及文件同步执行仓库 clang-format、尾部空白及 EOF 规范化。此次共 35 个文件，新增 2506 行、删除 2419 行；除头文件引用替换外，其余为格式调整。 sin 因此前发现的 UT 全局平台状态污染继续暂缓；experimental 文件继续分批处理。现有兼容转发头保留。 ## 关联的Issue 延续公共头文件重名问题的分阶段迁移：https://gitcode.com/cann/ops-math/issues/3243 ## 测试 在服务器 192.168.9.234 的 lcl_ops_9.0 容器（CANN 9.0.0 / aarch64）中，以基线 16700ac8d829f8e66180c612f45ed4384b51e2e0 创建独立工作区验证。35 个修改文件的 SHA256 与本地逐一一致。 - git diff --check：通过。 - pre-commit：所有 hook 通过；另显式扫描全部 35 文件的 OAT 检查通过。 - bash build.sh --opapi -j16：全量编译通过，生成 libopapi_math.so。 - 仓库 CI 脚本为涉及的 30 个算子生成 op_api 单测命令，指定 ascend910b 执行：605 个用例全部通过。 - 额外执行 fill_diagonal_v2,flatten,slogdet,svd 的 op_host 验证：容器中 libophost_math_ut.so 加载时缺少 ops::OpAICoreDef::AddConfig(const char*, ops::OpAICoreConfig&) 符号，导致注册失败及 FillDiagonalV2 InferShape 单测段错误。去掉全部 35 文件修改后的干净基线使用相同命令复现相同失败；该失败独立于本次头文件引用替换，op_host 验证尚未通过。 ## 文档更新 不涉及用户文档更新。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：公共头文件引用重构及格式规范化。 See merge request: cann/ops-math!6220	27 days ago
aclnn_maxn.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	22 days ago
maximum.cpp	fix: 修复 minimum/maximum 算子日志拼写/语法/copy-paste 错误 Co-authored-by: sun-yibo1<sunyibo1@hisilicon.com> # message auto-generated for no-merge-commit merge: !5188 merge fix/minmax-log-audit into master fix: 修复 minimum/maximum 算子日志拼写/语法/copy-paste 错误 Created-by: sun-yibo1 Commit-by: sun-yibo1 Merged-by: cann-robot Description: ## 描述 参照 log_audit_report.md，修复 ops-math 仓库极值类算子（minimum/maximum）的日志拼写、语法及 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 为拆分后的第 3 个，覆盖极值类（8 文件）。 ### 改动方法 仅修改日志文案/注释中的拼写、语法、copy-paste 错误，同时按 pre-commit clang-format 要求对文件做格式化。 ### 涉及修复（8 文件） **拼写/语法：** - only support → only supports（minimum/maximum） - must ge 1 / must >= 1 → must be >= 1（minimum/maximum onnx） **copy-paste/语义：** - minimum：Broadcast 第二个 self → other ## 关联的Issue - #2951 ## 测试 - 仅日志文案/格式修改，无逻辑变更，不影响编译与功能。 - 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!5188	2 months ago
maximum.h	支持neg等单目运算算子下一代kernel实现 Co-authored-by: 张磊<zhanglei121@huawei.com> # message auto-generated for no-merge-commit merge: !494 merge master into master 支持neg等单目运算算子下一代kernel实现 Created-by: zl_hw Commit-by: 张磊 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!494	10 months ago