| refactor: 迁移 arch35 算子 Reg API、修正拼写并规范化日志文案 Co-authored-by: esok11<yangsifa@huawei.com> # message auto-generated for no-merge-commit merge: !5135 merge f082601 into master refactor: 迁移 arch35 算子 Reg API、修正拼写并规范化日志文案 Created-by: esok11 Commit-by: esok11 Merged-by: cann-robot Description: ## 描述 对 arch35 架构下多个算子进行 AscendC Reg API 迁移、命名拼写修正及日志文案规范化,共 46 个文件,涉及 conversion/ 与 math/ 下 15 个算子。 主要改动: - AscendC Reg API 迁移(arch35 kernel 代码,21 个 .h 文件): - 地址空间修饰符:__local_mem__ → __ubuf__ - Reg 数据搬运 API 细化:加载用 AscendC::Reg::LoadAlign(原 Reg::DataCopy 加载语义),存储用 AscendC::Reg::StoreAlign(原 Reg::DataCopy 存储语义) - 掩码操作 API 重命名:AscendC::Reg::MaskNot → AscendC::Reg::Not、AscendC::Reg::MaskAnd → AscendC::Reg::And - 移除冗余 using AscendC::Reg::DataCopy; 声明 - 命名拼写修正(op_host/arch35,3 个文件): - 方法名 TilingForReleatedTranspose → TilingForRelatedTranspose(Releated→Related) - 成员变量 isReleatedTranspsoe_ → isRelatedTranspose_(Releated→Related, Transpsoe→Transpose) - 同步修改 confusion_transpose_d、depth_to_space 中对该方法的调用 - 日志清理(examples、op_host、op_api,22 个文件): - 失败路径日志级别 INFO → ERROR(Initialize ge failed、Run graph failed、Finalize ir graph session failed 等) - 文案语法修正:only support → only supports、is: %s and %s → are: %s and %s、expect %f → expected %f、multiples's → multiples' - tile 示例代码宏定义缩进格式化 ### 改动原因 配合 AscendC Reg API 演进,将 arch35 算子迁移至新的地址空间修饰符与 LoadAlign/StoreAlign/Not/And API;修正 transpose 相关命名中的拼写错误;统一日志级别与文案规范,提升日志可读性与问题定位效率。 ### 改动方法 - kernel 侧:逐文件替换 __local_mem__→__ubuf__、Reg::DataCopy(加载)→Reg::LoadAlign、Reg::DataCopy(存储)→Reg::StoreAlign、Reg::MaskNot→Reg::Not、Reg::MaskAnd→Reg::And,不改变控制流。 - host 侧:重命名 TilingForReleatedTranspose→TilingForRelatedTranspose、isReleatedTranspsoe_→isRelatedTranspose_,同步所有调用点。 - 日志:修改 printf/OP_LOGE/OP_LOGW/OP_LOGI 文案与级别字符串,不改变校验逻辑。 ## 关联的Issue - #2916 ## 测试 本 PR 为 API 迁移、命名修正与日志清理,通过已有测试用例(含 arch35 tiling UT 与 examples 二级冒烟)回归验证。 ## 文档更新 本 PR 未包含文档更新。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [x] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述:AscendC Reg API 迁移 + 命名拼写修正 + 日志规范化 See merge request: cann/ops-math!5135 | 1 个月前 |
| refactor: 规范化多算子日志文案与错误提示信息 Co-authored-by: esok11<yangsifa@huawei.com> # message auto-generated for no-merge-commit merge: !4924 merge f0821 into master refactor: 规范化多算子日志文案与错误提示信息 Created-by: esok11 Commit-by: esok11 Merged-by: cann-robot Description: ## 描述 对多个算子的日志文案与错误提示信息进行规范化清理,涉及 depth_to_space、matrix_set_diag_v2、pad_v3、pad_v3_grad_replicate、pad_v3_grad_replication、transpose 共 6 个算子、11 个文件。 主要改动: - 修复语法错误:must greater than → must be greater than、must less than → must be less than - 修复拼写错误:dimention → dimension、strat → start - 序数词规范化:-2th → -2nd、-1th → -1st - 补充缺失动词/介词:cannot empty → cannot be empty、Failed to vector size → Failed to get vector size - 统一用语:on aicore situation → in AICore situation - 增强日志信息量:部分校验失败日志补充实际的 dim num / padding size 数值,便于定位问题 - 统一错误提示描述:wrong gradOutput shape. → gradOutput shape does not match the expected shape derived from input and padding. - 格式化:transpose_onnx_plugin.cpp 中 REGISTER_CUSTOM_OP 缩进对齐 ### 改动原因 原有日志文案存在语法错误、拼写错误和表述不清晰的问题,影响问题定位效率和用户体验。本次统一规范化,提升日志的可读性和准确性。 ### 改动方法 逐文件修改 OP_LOGE/OP_LOGW/OP_LOGI 等日志调用中的文案字符串,不改变任何控制流和校验逻辑。transpose_onnx_plugin.cpp 额外对 REGISTER_CUSTOM_OP 调用做了缩进格式化。 ## 关联的Issue - #2786 ## 测试 本 PR 仅修改日志文案字符串,不涉及逻辑变更,通过已有测试用例回归验证。 ## 文档更新 本 PR 未包含文档更新。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码规范化/日志文案清理 See merge request: cann/ops-math!4924 | 1 个月前 |
| format: batch_to_space_nd 等算子代码格式化 Co-authored-by: zhanw_coding<zhanwang@huawei.com> # message auto-generated for no-merge-commit merge: !3822 merge fix-fmt into master format: batch_to_space_nd 等算子代码格式化 Created-by: zhanw_coding Commit-by: zhanw_coding Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> format: batch_to_space_nd 等算子代码格式化 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/2147 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码格式化 See merge request: cann/ops-math!3822 | 3 个月前 |
| ascend350 arch Co-authored-by: esok11<yangsifa@huawei.com> # message auto-generated for no-merge-commit merge: !5437 merge f0908 into master ascend350 arch Created-by: esok11 Commit-by: esok11 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!5437 | 1 个月前 |
| format: batch_to_space_nd 等算子代码格式化 Co-authored-by: zhanw_coding<zhanwang@huawei.com> # message auto-generated for no-merge-commit merge: !3822 merge fix-fmt into master format: batch_to_space_nd 等算子代码格式化 Created-by: zhanw_coding Commit-by: zhanw_coding Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> format: batch_to_space_nd 等算子代码格式化 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/2147 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码格式化 See merge request: cann/ops-math!3822 | 3 个月前 |
| refactor: 拆分 conversion 算子 infershape 与 tiling 大函数 Co-authored-by: esok11<yangsifa@huawei.com> # message auto-generated for no-merge-commit merge: !5256 merge b0831 into master refactor: 拆分 conversion 算子 infershape 与 tiling 大函数 Created-by: esok11 Commit-by: esok11 Merged-by: cann-robot Description: ## 描述 重构 conversion 目录下 4 个算子的 host 侧代码,将 infershape 与 arch35 tiling 中的大函数拆分为单一职责的小函数,降低圈复杂度、提升可读性与可维护性。改动不改变任何计算逻辑,仅为函数提取与职责拆分。 涉及算子与拆分点: - batch_to_space_nd:Inference() 拆分为 AppendBatchDim() / AppendSpatialDims() / AppendRemainDims() - broadcast_to(arch35 tiling):GetShapeInfo 拆分为 GetInputOutputShapes / WarnIfConstShapeMismatch / CheckDimNumLimits / CheckShapesValid / ReshapeAxes;GetUAxisInfo 拆分为 TrySetULastDim / TrySetUWholeFit / SetUAxisBySearch;CalcTensorSize4NBrwd 拆出 IsNBrwdFastPath / CalcNBrwdSlowTensorSize;CalcTensorSize 拆出 CalcIsDMABrcAFlag / AdjustMaxTensorSizeForUbBrc;CalcTilingKey 拆出 IsLastDimLargeB / IsUbBrcMode / IsUbBrcInnerCond - broadcast_to(infershape):BroadcastToInferShapeWithShapeTensor 拆分为 ValidateShapeDtype / DispatchGetConstValue / ResolveBroadcastDim - depth_to_space(arch35 tiling):ProcessShapeInfo 拆分为 SetInShapeNhwcDcr / SetInShapeNchwDcr / SetInShapeNhwcCrd / SetInShapeNchwCrd / BuildInShapeByFormatMode - transpose(arch35 tiling):AdjustUbCutAxisFactor 拆出 CalcUbAxesSizes / AdjustUbCutFactorSameAxis / AdjustUbCutFactorOutAxis / AdjustUbCutFactorInAxis;CalcUbAxisCutFactor 拆出 CalcUbCutFactorBothLeft / CalcUbCutFactorDiffLastAxis / CalcUbCutFactorOutLeftOnly / CalcUbCutFactorInLeftOnly;CalcUbAxesInfo 拆出 CompactUbAxes / CheckUbHwConstraint / MapUbCutPosToOut;CalcUbSplitInfo4MTE 拆出 CalcInSrcStrideConsecutive / CalcInSrcStrideDefault / CalcOutDstStride;AdjustInUbAxesPosition 拆出 ShiftInUbAxes4Borrow / AdjustUbPerm4BorrowCntTwo / AdjustUbPerm4BorrowCntThree / AdjustUbPermCntThreeNoOverlap / AdjustUbPermCntThreeOverlap;CalcBlockSplitInfo 拆出 CollectBlkAxes / CalcBlkUbCutAxis / CalcBlkAxesOffset ### 改动原因 原函数普遍较长(如 CalcUbSplitInfo、GetUAxisInfo、ProcessShapeInfo 等单函数超过 40 行且内含多个分支),圈复杂度高,难以阅读与维护。通过提取子函数使每个函数职责单一、命名自解释,并为关键函数补充 Doxygen 注释。 ### 改动方法 保持原有控制流与算式不变,将长函数体中的逻辑段落提取为 private/static 子函数,由原函数按原顺序调用。所有拆分均为等价提取,不引入新的分支或修改已有算式。 ## 关联的Issue - #2996 ## 测试 - 更新 depth_to_space arch35 单元测试(test_depth_to_space_tiling.cpp)的 expectTilingData 期望值,并移除遗留 TODO 注释。 - 重构不改变计算逻辑,已有 UT 覆盖行为不变。 ## 文档更新 无 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他,请描述:代码重构(函数拆分,降低圈复杂度) See merge request: cann/ops-math!5256 | 1 个月前 |
| ascend350 arch Co-authored-by: esok11<yangsifa@huawei.com> # message auto-generated for no-merge-commit merge: !5437 merge f0908 into master ascend350 arch Created-by: esok11 Commit-by: esok11 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!5437 | 1 个月前 |
| md大模型检测低错修改 Co-authored-by: gitcode-chenjiao<chenjiao31@huawei.com> # message auto-generated for no-merge-commit merge: !5183 merge master into master md大模型检测低错修改 Created-by: gitcode-chenjiao Commit-by: gitcode-chenjiao Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> md大模型检测低错修改 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> [# 2466](https://gitcode.com/cann/ops-math/issues/2446) ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ok ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> *.md ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [x] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!5183 | 1 个月前 |