Star0
0
Code
Code
Issues
Pull Requests
Actions
Discussion
Wiki
Analysis
Star0
0
cann-robotcann-robotim2col、matrix_diag、roll、grouped_bias_add_grad、reduce_var、reduce_std_v2 六个算子新增 ascend960dt 平台支持
a9a29898Add on 26 days agoHistory
FilesLast commitLast update
aclnn_roll.cpp
refactor: 为公共op_api头文件增加math后缀 Co-authored-by: clinglai0517<laichangling@huawei.com> # message auto-generated for no-merge-commit merge: !5839 merge refactor/math-prefix-common-headers into master refactor: 为公共op_api头文件增加math后缀 Created-by: clinglai0517 Commit-by: clinglai0517 Merged-by: cann-robot Description: ## 描述 ops-math/common/inc/op_api 中的 level2_base.h、level2_base_caculation.h、op_api_def.h 名称缺少仓库标识。在软件包合并交付或公共 include 目录集成时，存在被其他仓库同名文件覆盖，或因 include 路径顺序产生误引用的风险。 本 MR 参考同目录已有的 op_resource_math.h 命名，为三个公共头文件增加 _math 后缀，并通过兼容转发头分阶段迁移现有引用。 ## 修改内容 - 新增 level2_base_math.h、level2_base_caculation_math.h、op_api_def_math.h，作为带仓库标识的正式实现； - 原 level2_base.h、level2_base_caculation.h、op_api_def.h 保留为轻量兼容转发头，避免未迁移代码立即失效； - 新头文件之间的内部依赖统一使用 _math 后缀路径； - 更新 classify_rule.yaml，让 level2_base_math.h 继承原组件及 UT 覆盖率分类； - 将 89 处旧头文件引用迁移到新名称： - level2_base.h：20 处； - level2_base_caculation.h：9 处； - op_api_def.h：60 处； - 当前 MR 共修改 81 个文件，新增 434 行、删除 384 行。 为减少无关格式变化和后续合并冲突，本次未迁移会触发额外 clang-format、尾部空白或 EOF 修复，或组合触发既有 UT 隔离问题的文件。这些文件将拆分到后续 MR 逐步处理。 ## 剩余迁移 当前仓内仍有 67 处旧引用，分布在 65 个唯一文件中，继续通过兼容转发头工作： - level2_base.h：24 个文件； - level2_base_caculation.h：2 个文件； - op_api_def.h：41 个文件。 待旧引用分批迁移完成后，再结合实际打包和安装规则移除兼容转发头。本 MR 不移除旧头文件，也不声明已完成最终软件包去重。 ## 关联的 Issue 关联并在合入后关闭 #3243： https://gitcode.com/cann/ops-math/issues/3243 ## 测试 已完成： - 对本轮新增的 69 个纯引用替换文件逐行检查，80 组增删均仅修改目标 include，无其他代码或格式变化； - 额外纳入 aclnn_log10.cpp 和 aclnn_logdet.cpp，每个文件最终均为 3 行新增、3 行删除，低于 10 行修改阈值；math/sin/op_api/aclnn_sin.cpp 因聚合 UT 存在平台状态泄漏暂缓迁移； - 在 CANN 9.0.0 容器执行完整 pre-commit，尾部空白、EOF、clang-format、codespell 和 OAT 等检查全部通过； - 在 CANN 9.0.0 容器执行 bash build.sh --opapi -j32，全量构建通过； - 成功生成 build/libopapi_math.so，SHA-256：abaa8f8408979d931a8dad09f8e06875722fc42bbdddc72aaf9f14944bc163ea； - classify_rule.yaml 可正常解析，三个旧文件继续作为兼容转发头使用。 ## 文档更新 不涉及用户文档变更。 ## 类型标签 - [ ] Bug 修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他：代码重构 See merge request: cann/ops-math!5839
1 month ago
aclnn_roll.h
docs(roll): 补充空 dims 的参数长度例外 Co-authored-by: little_apple_12<020716ljh@gmail.com> # message auto-generated for no-merge-commit merge: !6104 merge fix/ops-math-2026-10-02-004 into master docs(roll): 补充空 dims 的参数长度例外 Created-by: little_apple_12 Commit-by: little_apple_12 Merged-by: cann-robot Description: ## 描述 aclnn_roll.h 和 aclnnRoll.md 的参数说明无条件要求 shifts 与 dims 等长；实际参数校验在 dims 为空时要求 shifts 恰有 1 个元素，空维路径按展平后的 tensor 执行 roll。照原参数说明传入两个空数组会被校验拒绝。本 PR 只同步公开文档，不改接口或运行逻辑。 逐 hunk 对账： 1. conversion/roll/op_api/aclnn_roll.h：同一处参数注释 hunk 为 shifts 和 dims 分别补充非空等长、空 dims 单 shift 与展平语义，并保留 dims 取值范围。 2. conversion/roll/docs/aclnnRoll.md 的 shifts 参数行：写明非空 dims 等长，空 dims 时 shifts 长度为 1 且沿展平 tensor 执行。 3. conversion/roll/docs/aclnnRoll.md 的 dims 参数行：保留原取值范围，并补充空 dims 的展平语义和单 shift 条件。 ### 关联的Issue 关联 issue #3388 ### 测试 真实执行命令： bash git diff --check && printf 'git diff --check: PASS\n' python3 -c 'from pathlib import Path; h=Path("conversion/roll/op_api/aclnn_roll.h").read_text(); d=Path("conversion/roll/docs/aclnnRoll.md").read_text(); c=Path("conversion/roll/op_api/aclnn_roll.cpp").read_text(); t=Path("conversion/roll/tests/ut/op_api/test_aclnn_roll.cpp").read_text(); checks=[("header non-empty dims length", "dims非空时，数组长度与dims保持一致" in h), ("header empty dims single shift + flatten", "dims为空时，数组长度必须为1" in h and "对展平后的x执行roll" in h), ("Markdown shifts exception", "dims为空时，数组长度必须为1，并对展平后的x执行roll" in d), ("Markdown dims exception", "dims为空时，对展平后的x执行roll，且shifts数组长度必须为1" in d), ("implementation rejects non-single empty-dims shifts", "dims->Size() == 0U && shifts->Size() != 1U" in c), ("existing UT covers empty dims with one shift", "test_empty_dims_single_shift" in t and "EXPECT_EQ(getWorkspaceResult, ACL_SUCCESS)" in t[t.index("test_empty_dims_single_shift"):])]; [print(("PASS" if ok else "FAIL")+": "+name) for name,ok in checks]; assert all(ok for _,ok in checks); print(str(len(checks))+" documentation/implementation checks passed")' 真实输出： text git diff --check: PASS PASS: header non-empty dims length PASS: header empty dims single shift + flatten PASS: Markdown shifts exception PASS: Markdown dims exception PASS: implementation rejects non-single empty-dims shifts PASS: existing UT covers empty dims with one shift 6 documentation/implementation checks passed 未运行 ACLNN/NPU 运行时 UT：本 PR 仅修改文档；没有声称执行运行时测试。 ### 文档更新 已更新 aclnn_roll.h 公共参数注释和 aclnnRoll.md 参数表，补充空 dims 时 shifts 必须恰为 1 且按展平 tensor 执行 roll 的说明。 ### 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [x] 文档更新 - [ ] 其他，请描述： ### 当前PR是否有AI参与: [x] 是 __1. Agent平台信息: Codex CLI 0.153.4 __2. 模型信息: gpt-5.6-luna __3. Prompt摘要: 围绕 issue #3388 复核 Roll 空 dims 参数约束，同步头文件与 Markdown 说明，逐 hunk 对账并验证文档与实现、现有 UT 一致。 ### 希望检视人员了解: 1. 贡献者提供已提交并已指派给 little_apple_12 的 issue #3388、专用 worktree 和提交流程约束；本轮行号、服务端 issue 状态、实现与 UT 依据及静态检查由 Codex 独立核验，未运行 NPU UT。 See merge request: cann/ops-math!6104
29 days ago
roll.cpp
im2col、matrix_diag、roll、grouped_bias_add_grad、reduce_var、reduce_std_v2 六个算子新增 ascend960dt 平台支持 Co-authored-by: liaohuming<liaohuming@huawei.com> # message auto-generated for no-merge-commit merge: !5844 merge master into master im2col、matrix_diag、roll、grouped_bias_add_grad、reduce_var、reduce_std_v2 六个算子新增 ascend960dt 平台支持 Created-by: liaohuming Commit-by: liaohuming Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 为 im2col、matrix_diag、roll、grouped_bias_add_grad、reduce_var、reduce_std_v2 六个算子新增 ascend960dt 平台支持，使其可在 ascend960dt（Ascend960DT_968B7）芯片上完成编译与调用。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/3614 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!5844
26 days ago
roll.h
reduce all Co-authored-by: lu-yukun<luyukun4@h-partners.com> # message auto-generated for no-merge-commit merge: !888 merge reduce_final into master reduce all Created-by: lu-yukun Commit-by: lu-yukun Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!888
9 months ago