已合并
整改安装重名头文件:权威 TLA / attention/common / mc2/common / math 侧保留;改次要副本 #10499
整改安装重名头文件:权威 TLA / attention/common / mc2/common / math 侧保留;改次要副本 #10499
已合并
chenyifan创建于 5 天前
chenyifan
chenyifan
5 天前

描述

安装重名头文件规范,完成 PR-3:其余算子 / 3rd 碰撞 / 3.7 类消歧(控变更文件数 < 200),消除安装包内跨算子同名头冲突。

方法:低风险只改次要副本——权威树 mc2/3rd/template_linear_algebraattention/commonmc2/common 等保留原名;次要侧加算子/组件前缀(kda_ / msa_ / gmm_ / mrs_v2_ / agm_v2_ / apace_ / wq_ / mararn_ 等)。同步更新 #include\file、include guard。无业务逻辑变更。禁止软链接。

安装分类规则(节选,本批相关)

# 场景 是否整改 处置
3.3 atvoss submodule 多份可见 不解决(归需屏蔽)
3.4 aclnn level2 双路径兼容 330 废弃(归需屏蔽)
3.7 transformer 误拷贝 math 本仓特化拷贝改名消歧(无法直接删改用 math)
3.9 仓内非对外同名拷贝 次要侧改名 / 去重

本批实际:3.9 + 3.7 消歧 + 同名不同实现亦改名;权威/公共树保留原名。

范围与结果

说明
范围 仍需整改清单中除 common.h / moe_common.h / moe_v2_common.h 外的成对/跨组件/TLA 次要副本
映射条数 43pr3_remap_batch1.tsv + pr3_remap_batch2.tsv)+ nd2nz 去重 + masked_select 改名
权威 TLA 包内多路径 dispatch_policy.hpp / tile_copy.hpp 等同组件备案,本批不改
变更文件数 196(< 200 硬约束)

命名约定

场景 约定
chunk_kda / msa 次要 kda_* / msa_*
matmul_reduce_scatter_v2 mrs_v2_*
all_gather_matmul_v2 agm_v2_*
gmm / apace / weight_quant gmm_* / apace_* / wq_*
matmul_all_reduce_add_rms_norm mararn_*
include guard 与文件名前缀一致(如 CATLASS_KDA_*
设备属性宏 保持 CATLASS_DEVICE 等原名(不随文件前缀改)

其它处置

处置
masked_select_v3.h moe_token_permute_masked_select_v3.h(算子特化,消歧 basename)
batch_mat_mul_nd2nz.h mat_mul_v3 副本,改为 include batch_mat_mul_v3 侧(内容相同)
sparse_flash_mla metadata root → *_kernel_metadata.h;arch22 → *_arch22_metadata.h

本分支变更统计

数量
变更文件 196
行变更 +375 / -797

关联的Issue

https://gitcode.com/cann/ops-transformer/issues/4356

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 chenyifan 的贡献)
chenyifanchenyifan
5 天前 创建了 pull request,commit 3a753f17
chenyifanchenyifan
5 天前 关联了issue:[Requirement|需求建议]: 仓内重名头文件过多需要整改
atomgit-bot
atomgit-bot
5 天前 评论:

🤖 正在生成合并请求摘要,请稍候…

likedislike
CANN-robotCANN-robot成员
5 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
5 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here.
You can get sig-info at here.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
attention cc-z, gaoxiang618 (2/2) cc-z, gaoxiang618 (2/1)
gmm kknan, chen-linxin4, cc-z (3/2) kknan, chen-linxin4, cc-z (3/1)
mc2 wang-minbo, yangzeheng (2/2) yangzeheng, wang-minbo (2/1)
moe chen-linxin4, 宋恺, cpy_123456, cc-z (4/2) chen-linxin4, 宋恺, cpy_123456, cc-z (4/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

mutex_lock, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
此处折叠了93条消息 查看更多
张海杰
张海杰成员
4 天前 评论:

/approve

likedislike
CANN-robotCANN-robot成员
4 天前 添加了label:api-check-failed
CANN-robotCANN-robot成员
4 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
4 天前 添加了label:ci-pipeline-passed
CANN-robotCANN-robot成员
4 天前 合入了pull request