已关闭
[重构] 更正 KthValue kernel common 同名头文件命名,区分 ops-math Sort 实现 #6002
黄晓彬创建于  13 天前关闭于  11 天前
黄晓彬成员
13 天前 创建

问题描述

ops-nn 的 index/kth_value/op_kernel/arch35/common/ 与 ops-math 的 math/sort/op_kernel/arch35/common/ 存在同名公共头文件。需要在 NN 仓对相关头文件更正命名,明确 KthValue 侧归属,避免跨仓维护和头文件引用时混淆。

当前本地核对的同名文件共 12 个:

  • merge_intra_core_base.h
  • merge_more_core_base.h
  • merge_sort_constants.h
  • non_last_small_axis_base.h
  • ping_pong_merge_sort.h
  • radix_more_core_base.h
  • radix_sort_constants.h
  • radix_sort_simd_utils.h
  • signed_zero_sort_utils.h
  • small_axis_insertion_base.h
  • small_axis_two_stage_base.h
  • util_type_simd.h

处理范围

  • 在 ops-nn 中为上述头文件采用统一、体现 KthValue 归属的命名(例如增加 kth_value_ 前缀);ops-math 文件名保持不变。
  • 同步更新 NN 仓所有相关 #include、构建配置和其他文件引用,覆盖 KthValue、Median、NanMedian 的共享调用。
  • 检查头文件保护宏,必要时同步改为与新文件名一致且不冲突的名称。
  • 本次仅调整命名及引用,保持算法、tiling、缓冲布局和同步逻辑不变。

验收建议

  • 扫描确认旧文件名引用无遗漏,无新增头文件名或保护宏冲突。
  • 完成受影响算子的编译与相关精度回归,确认重命名未改变实际选用的实现。
likedislike
黄黄晓彬成员
13 天前 修改了issue 的描述
yuning_chen
yuning_chen成员
13 天前 评论:

初步分析

感谢反馈,我们已开始核对这个问题。

现象理解:ops-nn 的 KthValue kernel common 头文件与 ops-math Sort 存在同名,希望在 NN 仓重命名并更新引用,保持算法与 tiling 不变。
当前判断:初步判断,待复现确认。这是命名与引用梳理诉求;所列 12 个文件是否仍同名,以及 Median/NanMedian 的共享引用范围,需对照当前目录核实。
涉及范围:index/kth_value/op_kernel/arch35/common/ 及 KthValue、Median、NanMedian 的头文件引用。
下一步:我们会核对同名文件与引用点,评估仅重命名及保护宏调整是否可行。

likedislike
黄黄晓彬成员
11 天前 关联了pull request:[修复] KthValue公共头文件命名及代码质量整改
黄黄晓彬成员
11 天前 关联了pull request:fix(index): address KthValue and NanMedian issues
CANN-robotCANN-robot成员
11 天前 关闭了 issue
CANN-robotCANN-robot成员
11 天前 添加了label:resolved