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

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

Files	Last commit	Last update
arch35	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	17 days ago
config	sort\|sort_with_index\|top_k_v2算子适配ascend960dt/ascend960pr Co-authored-by: tujun6<tujun6@huawei.com> # message auto-generated for no-merge-commit merge: !6202 merge add_960_sort into master sort\|sort_with_index\|top_k_v2算子适配ascend960dt/ascend960pr Created-by: tujun6 Commit-by: tujun6 Merged-by: cann-robot Description: ## 描述本 PR 为 sort/sort_with_index/top_k_v2 三个算子适配 ascend960dt/ascend960pr 平台，使其能够在这两个平台上编译和运行。主要改动： - 算子定义注册：在 `math/sort/op_host/sort_def.cpp`、`math/sort_with_index/op_host/sort_with_index_def.cpp`、`math/top_k_v2/op_host/top_k_v2_def.cpp` 中为 ascend960dt/ascend960pr 增加 AICore 配置（复用 ascend950 的 aicoreConfig） - 编译入口：三个算子的 `CMakeLists.txt` 中 `SUPPORT_COMPUTE_UNIT` / `SUPPORT_TILING_DIR` 增加 `ascend960dt`、`ascend960pr`（tiling 目录 arch35） - 二进制 kernel 配置：新增 `op_host/config/ascend960dt\|ascend960pr` 下的 `_binary.json` 与 `_simplified_key.ini` - 二进制编译注册：`scripts/kernel/binary_config/ascendc_config.json` 中扩展 Sort/SortWithIndex/TopKV2 的 `compute_units` 至 ascend960dt/ascend960pr，并补充对应编译选项 `-mllvm -cce-aicore-dcci-before-kernel-end=false` ## 关联的Issue https://gitcode.com/cann/ops-math/issues/3587 ## 测试 - 本地 pre-commit 全量检查通过（trailing-whitespace、end-of-file-fixer、check-json、clang-format、codespell 等全部 hook） - OAT 合规扫描通过（oat-py 1.0.3，PR 全部 19 个文件，License 头与文件类型违规均为 0） - 二进制 kernel 编译与算子精度验证依赖 CI 流水线及 ascend960 平台环境 ## 文档更新无 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6202	28 days ago
sort_def.cpp	sort\|sort_with_index\|top_k_v2算子适配ascend960dt/ascend960pr Co-authored-by: tujun6<tujun6@huawei.com> # message auto-generated for no-merge-commit merge: !6202 merge add_960_sort into master sort\|sort_with_index\|top_k_v2算子适配ascend960dt/ascend960pr Created-by: tujun6 Commit-by: tujun6 Merged-by: cann-robot Description: ## 描述本 PR 为 sort/sort_with_index/top_k_v2 三个算子适配 ascend960dt/ascend960pr 平台，使其能够在这两个平台上编译和运行。主要改动： - 算子定义注册：在 `math/sort/op_host/sort_def.cpp`、`math/sort_with_index/op_host/sort_with_index_def.cpp`、`math/top_k_v2/op_host/top_k_v2_def.cpp` 中为 ascend960dt/ascend960pr 增加 AICore 配置（复用 ascend950 的 aicoreConfig） - 编译入口：三个算子的 `CMakeLists.txt` 中 `SUPPORT_COMPUTE_UNIT` / `SUPPORT_TILING_DIR` 增加 `ascend960dt`、`ascend960pr`（tiling 目录 arch35） - 二进制 kernel 配置：新增 `op_host/config/ascend960dt\|ascend960pr` 下的 `_binary.json` 与 `_simplified_key.ini` - 二进制编译注册：`scripts/kernel/binary_config/ascendc_config.json` 中扩展 Sort/SortWithIndex/TopKV2 的 `compute_units` 至 ascend960dt/ascend960pr，并补充对应编译选项 `-mllvm -cce-aicore-dcci-before-kernel-end=false` ## 关联的Issue https://gitcode.com/cann/ops-math/issues/3587 ## 测试 - 本地 pre-commit 全量检查通过（trailing-whitespace、end-of-file-fixer、check-json、clang-format、codespell 等全部 hook） - OAT 合规扫描通过（oat-py 1.0.3，PR 全部 19 个文件，License 头与文件类型违规均为 0） - 二进制 kernel 编译与算子精度验证依赖 CI 流水线及 ascend960 平台环境 ## 文档更新无 ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6202	28 days ago
sort_infershape.cpp	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