ops-math/math/sort/op_host/arch35 · yuwang-/ops-math

cann-robotfix(sort,topkv2): replace deprecated invalid attribute logging macros

Files	Last commit	Last update
sort_tiling.cpp	sort/sort_with_index/top_k_v2/kth_value err msg整改 Co-authored-by: wuxiyuan<wuxiyuan@huawei.com> Co-authored-by: ConanHuang<huangxiaobin1@huawei.com> # message auto-generated for no-merge-commit merge: !3943 merge master into master sort/sort_with_index/top_k_v2/kth_value err msg整改 Created-by: wuxiyuan Commit-by: wuxiyuan;ConanHuang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 将 sort、sort_with_index、top_k_v2、kth_value 四个算子中的通用 OP_LOGE 调用替换为结构化错误日志宏，使错误信息包含参数名、实际值、期望值等结构化字段，便于问题定位。涉及算子 - sort - sort_with_index - top_k_v2 - kth_value 涉及文件 - math/kth_value/op_host/arch35/kth_value_tiling_arch35.cpp - math/kth_value/op_host/kth_value_infershape.cpp - math/sort/op_graph/sort_graph_infer.cpp - math/sort/op_host/arch35/sort_tiling.cpp - math/sort/op_host/arch35/sort_tiling_arch35.cpp - math/sort/op_host/arch35/sort_tiling_common.cpp - math/sort_with_index/op_host/arch35/sort_with_index_tiling.cpp - math/sort_with_index/op_host/sort_with_index_infershape.cpp - math/top_k_v2/op_graph/top_k_v2_graph_infer.cpp - math/top_k_v2/op_host/arch35/sort_with_index_tiling.h - math/top_k_v2/op_host/arch35/top_k_v2_tiling_arch35.cpp - math/top_k_v2/op_host/top_k_v2_infershape.cpp ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/2191 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> 已完成自验证 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 无 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：Err Msg整改 See merge request: cann/ops-math!3943	4 months ago
sort_tiling.h	sort cleancode处理 Co-authored-by: ConanHuang<huangxiaobin1@huawei.com> # message auto-generated for no-merge-commit merge: !4135 merge codecheck into master sort cleancode处理 Created-by: ConanHuang Commit-by: ConanHuang Merged-by: cann-robot Description: ## 描述 1. 魔法数字消除：2/3/4/6/2048 等字面量替换为 DOUBLE_BUFFER_NUM、MERGE_LIST_MAX_NUM、ONE_CORE_DATA_SIZE 等具名常量，涉及 17+ 处 2. 函数提取：BuildSortGraph()、SelectAndFinalizeKthValueRoute()、FillRadixKernelResources()、LaunchSortKernel() 等 6 个 helper 函数从主流程中提取，减少重复 3. sort_apt.cpp 重构：内核派发链由 if constexpr + return 改为 if/else if，拆出 LaunchRadixMoreCore/LaunchMergeSortRoute 模板 4. Lambda 捕获修复：[&] → [&info, &computeBatchNum]（sort_tiling_common.cpp:1168） 5. 格式化统一：const_cast<type> 间距、缩进（2→4空格）、注释对齐、文件末尾换行补全，涉及 30+ 文件 ## 关联的Issue 不涉及 ## 测试 1 sort sort_with_index top_k_v2 kth_value 全量ST通过 2 二级冒烟通过 ## 文档更新不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：格式和风格调整 See merge request: cann/ops-math!4135	3 months ago
sort_tiling_arch35.cpp	fix(sort,topkv2): replace deprecated invalid attribute logging macros Co-authored-by: wuxiyuan<wuxiyuan@huawei.com> # message auto-generated for no-merge-commit merge: !6638 merge master into master fix(sort,topkv2): replace deprecated invalid attribute logging macros Created-by: wuxiyuan Commit-by: wuxiyuan Merged-by: cann-robot Description: ## 描述 `OP_LOGE_WITH_INVALID_ATTR` 及其日志码将废弃。将 Sort、TopKV2 参数校验中的旧宏替换为 `OP_LOGE_FOR_INVALID_VALUE`，使用对应的参数值错误日志接口。 - Sort：替换 tiling 中 `axis` 校验的 1 处调用。 - TopKV2：替换 tiling 中 `sort_policy`、`dim` 以及 infershape 中 `dim` 校验的 3 处调用。共修改 3 个文件、4 处调用，保留原有校验条件、日志参数和错误返回逻辑。 ## 关联的Issue https://gitcode.com/cann/ops-math/issues/4018 ## 测试 - 对本次修改文件执行 `pre-commit run --files`，所有适用检查通过，包括 clang-format 和 OAT 合规检查。 - `git diff --check` 通过。 - 静态核查确认仅修改宏名称，Sort、TopKV2 目录中已无本次迁移涉及的六类旧宏调用。 - 二级冒烟通过 ## 文档更新无。 ## 类型标签 - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6638	18 days ago
sort_tiling_arch35.h	sort cleancode处理 Co-authored-by: ConanHuang<huangxiaobin1@huawei.com> # message auto-generated for no-merge-commit merge: !4135 merge codecheck into master sort cleancode处理 Created-by: ConanHuang Commit-by: ConanHuang Merged-by: cann-robot Description: ## 描述 1. 魔法数字消除：2/3/4/6/2048 等字面量替换为 DOUBLE_BUFFER_NUM、MERGE_LIST_MAX_NUM、ONE_CORE_DATA_SIZE 等具名常量，涉及 17+ 处 2. 函数提取：BuildSortGraph()、SelectAndFinalizeKthValueRoute()、FillRadixKernelResources()、LaunchSortKernel() 等 6 个 helper 函数从主流程中提取，减少重复 3. sort_apt.cpp 重构：内核派发链由 if constexpr + return 改为 if/else if，拆出 LaunchRadixMoreCore/LaunchMergeSortRoute 模板 4. Lambda 捕获修复：[&] → [&info, &computeBatchNum]（sort_tiling_common.cpp:1168） 5. 格式化统一：const_cast<type> 间距、缩进（2→4空格）、注释对齐、文件末尾换行补全，涉及 30+ 文件 ## 关联的Issue 不涉及 ## 测试 1 sort sort_with_index top_k_v2 kth_value 全量ST通过 2 二级冒烟通过 ## 文档更新不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [x] 其他，请描述：格式和风格调整 See merge request: cann/ops-math!4135	3 months ago
sort_tiling_common.cpp	perf(sort): 优化归并、非末轴排序及索引输出 Co-authored-by: ConanHuang<huangxiaobin1@huawei.com> # message auto-generated for no-merge-commit merge: !5672 merge master into master perf(sort): 优化归并、非末轴排序及索引输出 Created-by: ConanHuang Commit-by: ConanHuang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 优化 Sort 在多行归并、非末轴小轴及 UB 可容纳整行的大轴场景中的执行开销，保持稳定排序、升降序及 values/indices 语义。 - 引入公共 ping-pong 归并，减少阶段间回拷；在适用路径批量归并多行，并优化多核末轮归并。 - 按实际 UB 布局统一选择整行归并路径，优化非末轴批次规划、NDDMA 搬运与向量化索引输出。 - 对 RegBase 平台排序轴后全为单例维的输入，通过 Reshape 消除冗余转置；其他输入沿用原路径。 - 同步 TopKV2 依赖模板，补充 NaN、正负零、尾块及稳定索引看护。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/3230 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> - Ascend950PR_9579 ：Sort kernel 395 条、正负零逐位比较 18 条、Sort ACLNN 40 条、TopKV2 受影响用例 177 条，合计 630 条精度全部通过，Sort 同时比较 values 和 indices。 - 相对前一版已验证实现，设备耗时异常项复测后未确认稳定超过 5% 且超过 0.5 μs 的劣化 - 二级冒烟通过 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [x] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!5672	1 month ago
sort_tiling_common.h	perf(sort): 优化归并、非末轴排序及索引输出 Co-authored-by: ConanHuang<huangxiaobin1@huawei.com> # message auto-generated for no-merge-commit merge: !5672 merge master into master perf(sort): 优化归并、非末轴排序及索引输出 Created-by: ConanHuang Commit-by: ConanHuang Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 优化 Sort 在多行归并、非末轴小轴及 UB 可容纳整行的大轴场景中的执行开销，保持稳定排序、升降序及 values/indices 语义。 - 引入公共 ping-pong 归并，减少阶段间回拷；在适用路径批量归并多行，并优化多核末轮归并。 - 按实际 UB 布局统一选择整行归并路径，优化非末轴批次规划、NDDMA 搬运与向量化索引输出。 - 对 RegBase 平台排序轴后全为单例维的输入，通过 Reshape 消除冗余转置；其他输入沿用原路径。 - 同步 TopKV2 依赖模板，补充 NaN、正负零、尾块及稳定索引看护。 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> https://gitcode.com/cann/ops-math/issues/3230 ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> - Ascend950PR_9579 ：Sort kernel 395 条、正负零逐位比较 18 条、Sort ACLNN 40 条、TopKV2 受影响用例 177 条，合计 630 条精度全部通过，Sort 同时比较 values 和 indices。 - 相对前一版已验证实现，设备耗时异常项复测后未确认稳定超过 5% 且超过 0.5 μs 的劣化 - 二级冒烟通过 ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> 不涉及 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [x] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!5672	1 month ago