| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
cleancode降低代码重复率 Co-authored-by: jisongyuan<jisongyuan@h-partners.com> # message auto-generated for no-merge-commit merge: !5538 merge master into master cleancode降低代码重复率 Created-by: ji-songyuan Commit-by: jisongyuan Merged-by: cann-robot Description: ## 描述 本 PR 对 conversion 和 random 模块下多个算子的源文件进行 cleancode 顺序调整,共涉及 8 个文件,改动量 +51/-51,旨在降低 cleancode 扫描的代码重复率。 主要解决以下 cleancode 重复率偏高的问题: - **include 头文件顺序混乱**:多个 .cpp 文件的 #include 顺序无规律,与仓内其他文件的相似度计算偏高 - **常量/变量声明顺序混乱**:如 pad_v4_grad_tiling.cpp 中 constexpr 与 static map 交错排列 - **参数检查顺序不一致**:如 aclnn_pad2d_backward.cpp 的 CheckNotNull / CheckDtypeValid 中各检查项顺序与同类算子不一致 cleancode 扫描基于行级相似度,上述顺序差异会被误判为重复代码块,统一调整声明/检查顺序可降低重复率。 ### 改动明细 | # | 文件 | 改动 | 说明 | |---|------|------|------| | 1 | conversion/chunk_cat/op_api/aclnn_chunk_cat.cpp | +5/-5 | include 顺序调整:按 op_api_def/aclnn_check/opdev/common/opdev 分组排序 | | 2 | conversion/concat_d/op_api/aclnn_cat.cpp | +11/-11 | include 顺序调整:同上分组排序 | | 3 | conversion/flatten/op_host/op_api/flatten.cpp | +3/-3 | include 顺序调整:按 make_op_executor/op_def/aicpu_task/op_dfx/op_log/op_executor/platform/shape_utils 排序 | | 4 | conversion/pad_v3_grad_replicate/op_host/op_api/aclnn_pad2d_backward.cpp | +9/-8 | CheckNotNull 调整为 self→gradOutput→gradInput→padding;CheckDtypeValid 调整为 gradInput→gradOutput→self | | 5 | conversion/pad_v4_grad/op_host/pad_v4_grad_tiling.cpp | +6/-6 | 常量声明顺序调整:按 map→TILING_KEY→shape 上限→SMALL_W_LIMIT 分组排列 | | 6 | random/drop_out_do_mask/op_api/aclnn_dropout_backward.cpp | +8/-8 | include 顺序调整:按 platform→common_types→format_utils→op_executor→data_type_utils→op_log→tensor_view_utils→op_dfx→shape_utils 排序 | | 7 | random/drop_out_do_mask/op_api/aclnn_dropout_do_mask.cpp | +5/-5 | include 顺序调整 + 常量顺序:include 同上分组;BIT_NUMBER 与 UINT8_BIT_NUMBER 顺序对换 | | 8 | random/dsa_gen_bit_mask/op_host/op_api/aclnn_dropout.cpp | +4/-4 | include 顺序调整:按 common_types→format_utils→data_type_utils→op_dfx→shape_utils→op_executor→platform→op_log→tensor_view_utils 排序 | ### 改动原则 - **纯顺序调整,不改变任何代码逻辑**:所有改动均为 include 顺序、常量声明顺序、参数检查顺序的调整,不涉及函数逻辑、控制流、变量名等 - **对齐实现**:调整后的顺序与同模块其他算子保持一致 - **聚焦最小改动**:仅调整顺序以降低 cleancode 重复率,不做额外的重构 ## 关联的Issue 关联 Issue:https://gitcode.com/cann/ops-math/issues/3118 ## 测试 - 顺序调整不改变代码逻辑,通过人工 review 验证语义等价 - 改动不涉及算子行为,CI 编译与功能测试结果不受影响 - cleancode 扫描重复率预计下降(需重新扫描确认) ## 文档更新 无文档更新,本 PR 仅涉及源代码 (.cpp) 的顺序调整。 ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他:cleancode 重复率修复 ## AI/Agent生成声明 - [ ] AI辅助编写 See merge request: cann/ops-math!5538 | 1 个月前 | |
fix clean code Co-authored-by: wy519<wangyang519@huawei.com> # message auto-generated for no-merge-commit merge: !1871 merge fix_clean_code into master fix clean code Created-by: wy519 Commit-by: wy519 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 修正cleanCode问题 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!1871 | 7 个月前 | |
fix clean code Co-authored-by: wy519<wangyang519@huawei.com> # message auto-generated for no-merge-commit merge: !1871 merge fix_clean_code into master fix clean code Created-by: wy519 Commit-by: wy519 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 修正cleanCode问题 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!1871 | 7 个月前 | |
新增ChunkCat算子 Co-authored-by: wy519<wangyang519@huawei.com> # message auto-generated for no-merge-commit merge: !1706 merge chunk_cat into master 新增ChunkCat算子 Created-by: wy519 Commit-by: wy519 Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> 新增ChunkCat算子 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/1053 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> 测试600个case,精度性能均无问题 ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> 新增README.md和aclnnChunkCat文件 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-math!1706 | 7 个月前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 1 个月前 | ||
| 7 个月前 | ||
| 7 个月前 | ||
| 7 个月前 |