| docs_fix Co-authored-by: jisongyuan@h-partners.com<jisongyuan@h-partners.com> # message auto-generated for no-merge-commit merge: !9894 merge master into master docs_fix Created-by: ji-songyuan Commit-by: jisongyuan@h-partners.com Merged-by: cann-robot Description: ## 描述 本 PR 对 ctc_loss、l1_loss、fused_adamw、fused_sgd 四个算子的接口文档(.md)进行勘误修复,共涉及 4 个文件,改动量 +15/-16。 主要解决以下文档与实现不一致的问题: - **参数名大小写不一致**:文档中参数名与实际接口签名不符 - **描述对象错误**:文档描述的作用对象与实际接口语义不匹配 - **约束条件过时**:文档约束与当前实现逻辑不符 - **参数命名规范**:文档参数名未遵循 camelCase 规范 ### 改动明细 | # | 文件 | 改动 | 说明 | |---|------|------|------| | 1 | loss/ctc_loss_v2/docs/aclnnCtcLossV2.md | 多处 | 将 targetLengths 统一改为 targetlengths,与实际 aclIntArray* 参数命名一致,消除大小写歧义 | | 2 | loss/lp_loss/docs/aclnnL1Loss.md | +2/-2 | 修正 target 参数描述:broadcast 关系对象由 gradOutput、target 更正为 self;补充 out 参数说明为"输出 tensor,存放 L1Loss 计算结果",并将"shape 只支持 0 维"规范为"out 为 0 维 tensor" | | 3 | optim/fused_adamw/docs/aclnnFusedAdamw.md | +1/-1 | 修正约束描述:原"数据格式不在支持范围"更正为"数据类型不同",与实际类型校验逻辑一致 | | 4 | optim/fused_sgd/docs/aclnnFusedSgd.md | +1/-2 | 参数名 workspace_size 更正为 workspaceSize,与 aclnnFusedSgd 接口签名一致;删除过时的 gradScale 等于 0 约束说明 | ### 改动原则 - **纯文档修复,不改变任何代码逻辑**:所有改动均为 .md 文件内的文字描述,不涉及 .cpp/.h 源文件 - **对齐实现**:修正后的描述与当前算子实现、接口签名、约束逻辑保持一致 - **聚焦最小改动**:仅修正文档与实现不一致项,不做额外的文档重构 ## 关联的Issue Issue #5554 ## 测试 - 文档改动无需编译/功能测试,通过人工 review 验证描述与代码实现一致 - 改动不影响算子行为,CI 编译与功能测试结果不受影响 ## 文档更新 本 PR 即为文档更新,涉及以下文件: - loss/ctc_loss_v2/docs/aclnnCtcLossV2.md - loss/lp_loss/docs/aclnnL1Loss.md - optim/fused_adamw/docs/aclnnFusedAdamw.md - optim/fused_sgd/docs/aclnnFusedSgd.md ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [x] 文档更新 - [ ] 其他,请描述: ## AI/Agent生成声明 - [ ] AI辅助编写 See merge request: cann/ops-nn!9894 | 27 天前 |
| format cpp Co-authored-by: yang-di52<yangdi52@huawei.com> # message auto-generated for no-merge-commit merge: !6784 merge issue_fix into master format cpp Created-by: yang-di52 Commit-by: yang-di52 Merged-by: cann-robot Description: ## 描述 批量刷新cpp代码格式 ## 关联的Issue [#3791](https://gitcode.com/cann/ops-nn/issues/3791) ## 测试 <!--描述进行了哪些测试来验证你的改动。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码格式化 ## AI/Agent生成声明 <!-- [x] 表示选中 --> - [ ] AI辅助编写 See merge request: cann/ops-nn!6784 | 2 个月前 |
| 【bugfix】修复空指针core dump问题 Co-authored-by: majiajian_hw<majiajian@huawei.com> # message auto-generated for no-merge-commit merge: !8598 merge bugfix_fused_sgd0812 into master 【bugfix】修复空指针core dump问题 Created-by: nunnons2 Commit-by: majiajian_hw Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 修复fusedSgd算子空指针core dump问题 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #000--> https://gitcode.com/cann/ops-nn/issues/4739 ## 测试 <!--描述进行了哪些测试来验证你的改动。--> 已完成 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: ## AI/Agent生成声明 <!-- [x] 表示选中 --> - [ ] AI辅助编写 See merge request: cann/ops-nn!8598 | 1 个月前 |
| 【特性】新增fusedSgd Co-authored-by: majiajian_hw<majiajian@huawei.com> # message auto-generated for no-merge-commit merge: !6220 merge add_fused_sgd_2 into master 【特性】新增fusedSgd Created-by: nunnons2 Commit-by: majiajian_hw Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 新增fusedSgd算子 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #000--> https://gitcode.com/cann/ops-nn/issues/3454 ## 测试 <!--描述进行了哪些测试来验证你的改动。--> 已完成泛化测试 #### 测试用例 std::vector<float> paramsRefHostData1 = {1, 2, 3, 4, 5, 6, 7, 8}; std::vector<float> gradsRefHostData1 = {0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8}; std::vector<float> momentumHostData1 = {0, 0, 0, 0, 0, 0, 0, 0}; std::vector<float> paramsRefHostData2 = {9, 10, 11, 12}; std::vector<float> gradsRefHostData2 = {0.9, 1.0, 1.1, 1.2}; std::vector<float> momentumHostData2 = {0, 0, 0, 0}; std::vector<float> gradScaleOptionalHostData = {1.0}; std::vector<int64_t> inputShape1 = {2, 2, 2}; std::vector<int64_t> inputShape2 = {2, 2}; std::vector<int64_t> scalarShape = {1}; #### 执行结果 ====== Tensor 2 gradsRef results ====== result[0] is: 0.900000 result[1] is: 1.000000 result[2] is: 1.100000 result[3] is: 1.200000 ------ Momentum buffer 2 ------ result[0] is: 0.990000 result[1] is: 1.100000 result[2] is: 1.210000 result[3] is: 1.320000 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> 新增README.md、aclnnFusedSgd.md文件 ## 类型标签 <!-- [x] 表示选中 --> - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-nn!6220 | 3 个月前 |
| clean_code Co-authored-by: jisongyuan<jisongyuan@h-partners.com> # message auto-generated for no-merge-commit merge: !9785 merge master into master clean_code Created-by: ji-songyuan Commit-by: jisongyuan Merged-by: cann-robot Description: ## 描述 本 PR 对 fused_sgd、fused_adamw、sigmoid_cross_entropy_with_logits_grad_v2 三个算子的 host 侧代码进行 clean code 修复,共涉及 4 个文件,改动量 +10/-9。 主要解决以下 cleancode 规则告警: - **override 缺失**:重写基类虚函数时未明确指定 override,无法在编译期校验重写正确性 - **const 正确性**:不修改对象状态的成员函数未加 const 修饰,无法被 const 对象调用 - **变量命名歧义**:lambda 捕获参数 name 与宏内同名变量存在混淆风险 - **格式与文件结尾**:初始化列表格式不规范、源文件末尾缺失换行符 ### 改动明细 | # | 文件 | 改动 | 说明 | |---|------|------|------| | 1 | loss/sigmoid_cross_entropy_with_logits_grad_v2/op_host/arch35/sigmoid_cross_entropy_with_logits_grad_v2_arch35_tiling.h | +1/-1 | DumpTilingInfo() 声明加 override,使编译器能在编译期检查虚函数重写签名匹配,避免无意中隐藏基类同名函数 | | 2 | optim/fused_adamw/op_host/fused_adamw_tiling.cpp | +5/-4 | 将 checkDtype/checkShape lambda 的 name 参数重命名为 nameOfOps,消除与宏内同名变量的歧义;调整 OP_LOGE_FOR_INVALID_DTYPES_WITH_REASON 调用格式,参数逐行对齐提升可读性 | | 3 | optim/fused_sgd/op_host/fused_sgd_tiling.cpp | +1/-1 | FusedSgdTiling::SetTilingData 加 const 修饰。该函数仅读取 this 成员、写入传入的 tilingData 参数,符合 const 成员函数语义 | | 4 | optim/fused_sgd/op_host/fused_sgd_tiling.h | +3/-3 | 构造函数初始化列表格式规整(context_(context){}; → context_(context) {};);SetTilingData 声明加 const 与定义对齐;文件末尾补换行符消除 No newline at end of file 警告 | ### 改动原则 - **纯代码风格/语义增强,不改变任何运行时行为**:所有改动均为编译期可验证的语义提示(override/const)或格式规整,生成的机器码与改动前等价 - **向后兼容**:函数签名增加 const/override 不影响既有调用方,非 const 对象仍可调用 const 成员函数 - **聚焦最小改动**:仅修正 cleancode 扫描告警项,不做额外的重构或逻辑调整 ## 关联的Issue Issue #5445。本 PR 为独立 cleancode 修复,不针对特定功能缺陷。 ## 测试 - 依赖 CI 流水线编译验证:当前 PR 已通过 ci-pipeline-passed 检查 - 改动均为声明/格式层面,不涉及运行时逻辑,无需额外功能测试 - override/const 在编译期生效,若基类签名变更或函数误写为非 const,编译器会直接报错 ## 文档更新 无需文档更新。本 PR 不引入新接口、不改变算子行为,对外 API 与 tiling 协议保持不变。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:Clean Code 规则修复(override/const/命名/格式) ## AI/Agent生成声明 - [ ] AI辅助编写 See merge request: cann/ops-nn!9785 | 28 天前 |
| clean_code 代码重复率整改 Co-authored-by: zhuzemao<zhuzemao1@h-partners.com> # message auto-generated for no-merge-commit merge: !8694 merge clean_code into master clean_code 代码重复率整改 Created-by: zhuzemao Commit-by: zhuzemao Merged-by: cann-robot Description: ## 描述 clean_code 代码重复率整改 ## 关联的Issue https://gitcode.com/cann/ops-nn/issues/4802 ## 测试 不涉及 ## 文档更新 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述: ## AI/Agent生成声明 <!-- [x] 表示选中 --> - [ ] AI辅助编写 See merge request: cann/ops-nn!8694 | 1 个月前 |
| format cpp Co-authored-by: yang-di52<yangdi52@huawei.com> # message auto-generated for no-merge-commit merge: !6784 merge issue_fix into master format cpp Created-by: yang-di52 Commit-by: yang-di52 Merged-by: cann-robot Description: ## 描述 批量刷新cpp代码格式 ## 关联的Issue [#3791](https://gitcode.com/cann/ops-nn/issues/3791) ## 测试 <!--描述进行了哪些测试来验证你的改动。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码格式化 ## AI/Agent生成声明 <!-- [x] 表示选中 --> - [ ] AI辅助编写 See merge request: cann/ops-nn!6784 | 2 个月前 |
| 【特性】新增fusedSgd Co-authored-by: majiajian_hw<majiajian@huawei.com> # message auto-generated for no-merge-commit merge: !6220 merge add_fused_sgd_2 into master 【特性】新增fusedSgd Created-by: nunnons2 Commit-by: majiajian_hw Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 新增fusedSgd算子 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #000--> https://gitcode.com/cann/ops-nn/issues/3454 ## 测试 <!--描述进行了哪些测试来验证你的改动。--> 已完成泛化测试 #### 测试用例 std::vector<float> paramsRefHostData1 = {1, 2, 3, 4, 5, 6, 7, 8}; std::vector<float> gradsRefHostData1 = {0.1, 0.2, 0.3, 0.4, 0.5, 0.6, 0.7, 0.8}; std::vector<float> momentumHostData1 = {0, 0, 0, 0, 0, 0, 0, 0}; std::vector<float> paramsRefHostData2 = {9, 10, 11, 12}; std::vector<float> gradsRefHostData2 = {0.9, 1.0, 1.1, 1.2}; std::vector<float> momentumHostData2 = {0, 0, 0, 0}; std::vector<float> gradScaleOptionalHostData = {1.0}; std::vector<int64_t> inputShape1 = {2, 2, 2}; std::vector<int64_t> inputShape2 = {2, 2}; std::vector<int64_t> scalarShape = {1}; #### 执行结果 ====== Tensor 2 gradsRef results ====== result[0] is: 0.900000 result[1] is: 1.000000 result[2] is: 1.100000 result[3] is: 1.200000 ------ Momentum buffer 2 ------ result[0] is: 0.990000 result[1] is: 1.100000 result[2] is: 1.210000 result[3] is: 1.320000 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> 新增README.md、aclnnFusedSgd.md文件 ## 类型标签 <!-- [x] 表示选中 --> - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-nn!6220 | 3 个月前 |
| 【bugfix】fusedSgd算子支持950 Co-authored-by: majiajian_hw<majiajian@huawei.com> # message auto-generated for no-merge-commit merge: !7877 merge add_fusedSgd0723 into master 【bugfix】fusedSgd算子支持950 Created-by: nunnons2 Commit-by: majiajian_hw Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 该 PR 为 fusedSgd 融合 SGD 算子新增了 Ascend 950(950PR/950DT)硬件平台的支持,包括芯片配置注册、平台二进制配置文件,以及文档中支持状态的更新。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #000--> https://gitcode.com/cann/ops-nn/issues/4432 ## 测试 <!--描述进行了哪些测试来验证你的改动。--> 已完成 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> 更新了README.md、aclnnFusedSgd.md文档 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: ## AI/Agent生成声明 <!-- [x] 表示选中 --> - [x] AI辅助编写 See merge request: cann/ops-nn!7877 | 2 个月前 |