已合并
aclnn文件加上芯片标签 #7981
zhouwenfang创建于 7月27日
aclnn文件加上芯片标签 #7981
已合并
zhouwenfang创建于 7月27日
zhouwenfang
zhouwenfang成员
7月27日

描述

aclnn文件加上芯片标签

关联的Issue

https://gitcode.com/cann/ops-nn/issues/4354

测试

NA

文档更新

quant目录下的所有aclnn.md

类型标签

  • [x ] 文档更新

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 zhouwenfang 的贡献)
zhouwenfangzhouwenfang成员
7月27日 创建了 pull request,commit 09f449c5
atomgit-bot
atomgit-bot
7月27日 评论:

变更摘要

本次PR为 quant/ 目录下所有 aclnn.md 算子文档统一添加了芯片支持标签,明确标注每个算子所支持的昇腾芯片型号(如 Ascend310P、Ascend910B、Atlas A2 训练系列等),共涉及44个文档文件的更新。

主要改动

  • 为 aclnn.md 文档添加芯片支持声明:在每个算子的 aclnn.md 文件顶部新增 # 支持芯片: 标签,列出该算子兼容的昇腾芯片型号,例如 Ascend310P、Ascend910B、Atlas A2 训练系列 等,方便用户快速识别算子芯片适配范围。
  • 芯片标签覆盖多种配置:不同算子支持不同芯片组合,部分算子仅支持 Ascend310P,部分支持 Ascend310P 与 Ascend910B 和 Atlas A2,少数算子额外涵盖 Ascend910 Premium A、Ascend910 A、Ascend910 Pro B、Ascend910 Premium B、Ascend910B2/B3/B4 等型号。
  • 统一 quant/ 目录下全部 aclnn.md 文档:44个算子子目录(如 Abs、Add、BatchMatMul、Conv2D 等)下的 aclnn.md 均完成标签添加,文档格式保持一致。
likedislike
不准确?
atomgit-bot
atomgit-bot
7月27日 评论:

代码审查

审查总结

我已完成对所有 44 个变更文件的逐一审查。以下是审查结果汇总:

发现的问题

  • P1(3 个):产品支持状态值在格式化过程中被错误修改

    • quant/ascend_quant_v2/docs/aclnnAscendQuant.md:Atlas 推理系列产品(310p) 从"支持"误改为"不支持",且与文件中 310p 约束说明矛盾;Ascend 950PR/Ascend 950DT 从"不支持"误改为"支持"
    • quant/ascend_anti_quant_v2/docs/aclnnAscendAntiQuant.md:Atlas 推理系列产品(310p) 从"支持"误改为"不支持",且与文件中 310p 约束说明矛盾
  • P3(2 个):文件末尾缺少换行符

    • quant/dynamic_block_quant/docs/aclnnDynamicBlockQuant.md
    • quant/dynamic_block_quant/docs/aclnnDynamicBlockQuantV2.md

各文件审查结果

文件 结果
docs/zh/op_list.md 无问题
quant/anti_mx_quant/docs/aclnnAntiMxQuant.md 无问题
quant/ascend_anti_quant_v2/docs/aclnnAscendAntiQuant.md P1:310p 支持状态错误
quant/ascend_quant_v2/docs/aclnnAscendQuant.md P1:950 和 310p 支持状态错误
quant/ascend_quant_v2/docs/aclnnAscendQuantV3.md 无问题
quant/dequant_bias/docs/aclnnDequantBias.md 无问题
quant/dequant_swiglu_quant/docs/aclnnDequantSwigluQuant.md 无问题
quant/dequant_swiglu_quant/docs/aclnnDequantSwigluQuantV2.md 无问题
quant/dynamic_block_mx_quant/docs/aclnnDynamicBlockMxQuant.md 无问题
quant/dynamic_block_quant/docs/aclnnDynamicBlockQuant.md P3:缺少末尾换行符
quant/dynamic_block_quant/docs/aclnnDynamicBlockQuantV2.md P3:缺少末尾换行符
quant/dynamic_dual_level_mx_quant/docs/aclnnDynamicDualLevelMxQuant.md 无问题
quant/dynamic_mx_quant/docs/aclnnDynamicMxQuant.md 无问题
quant/dynamic_mx_quant/docs/aclnnDynamicMxQuantV2.md 无问题
quant/dynamic_mx_quant/docs/aclnnDynamicMxQuantV3.md 无问题
quant/dynamic_mx_quant_with_dual_axis/docs/aclnnDynamicMxQuantWithDualAxis.md 无问题
quant/dynamic_mx_quant_with_dual_axis/docs/aclnnDynamicMxQuantWithDualAxisV2.md 无问题
quant/dynamic_quant/docs/aclnnDynamicQuant.md 无问题
quant/dynamic_quant/docs/aclnnDynamicQuantV3.md 无问题
quant/dynamic_quant/docs/aclnnDynamicQuantV4.md 无问题
quant/dynamic_quant_v2/docs/aclnnDynamicQuantV2.md 无问题
quant/fake_quant_affine_cachemask/docs/aclnnFakeQuantPerChannelAffineCachemask.md 无问题
quant/fake_quant_affine_cachemask/docs/aclnnFakeQuantPerTensorAffineCachemask.md 无问题
quant/flat_quant/docs/aclnnFlatQuant.md 无问题
quant/flat_quant/docs/aclnnFlatQuantV2.md 无问题
quant/flat_quant/docs/aclnnFlatQuantV3.md 无问题
quant/group_quant/docs/aclnnGroupQuant.md 无问题
quant/grouped_dynamic_block_quant/docs/aclnnGroupedDynamicBlockQuant.md 无问题
quant/grouped_dynamic_block_quant/docs/aclnnGroupedDynamicBlockQuantV2.md 无问题
quant/grouped_dynamic_mx_quant/docs/aclnnGroupedDynamicMxQuant.md 无问题
quant/grouped_dynamic_mx_quant/docs/aclnnGroupedDynamicMxQuantV2.md 无问题
quant/grouped_dynamic_mx_quant_with_dual_axis/docs/aclnnGroupedDynamicMxQuantWithDualAxis.md 无问题
quant/grouped_quant_max/docs/aclnnGroupedQuantMax.md 无问题
quant/mx_to_block_mx_quant/docs/aclnnMxToBlockMxQuant.md 无问题
quant/quant_max/docs/aclnnQuantMax.md 无问题
quant/quantize/docs/aclnnQuantize.md 无问题
quant/swi_glu_quant/docs/aclnnSwiGluQuant.md 无问题
quant/swi_glu_quant/docs/aclnnSwiGluQuantV2.md 无问题
quant/swiglu_backward_mx_quant_with_dual_axis/docs/aclnnSwigluBackwardMxQuantWithDualAxis.md 无问题
quant/swiglu_mx_quant/docs/aclnnSwigluMxQuant.md 无问题
quant/swiglu_mx_quant_with_dual_axis/docs/aclnnSwigluMxQuantWithDualAxis.md 无问题
quant/trans_quant_param/docs/aclnnTransQuantParam.md 无问题
quant/trans_quant_param_v2/docs/aclnnTransQuantParamV2.md 无问题
quant/trans_quant_param_v2/docs/aclnnTransQuantParamV3.md 无问题

整体风险评估

中等风险。大部分文件的芯片标签添加工作正确无误,但 aclnnAscendQuant.md 和 aclnnAscendAntiQuant.md 两个文件在格式化过程中出现了产品支持状态值的错误修改(P1),会导致用户对芯片支持情况产生错误认知,建议在合入前修正。末尾换行符缺失(P3)为低影响问题。

类型 数量
🔴 阻塞 3
🟡 建议 1

⛔ 需要修改

likedislike
不准确?
CANN-robotCANN-robot成员
7月27日 添加了label:stat/needs-squash
CANN-robotCANN-robot成员
7月27日 添加了label:cann-cla/yes
此处折叠了56条消息 查看更多
CANN-robotCANN-robot成员
7月27日 添加了label:lgtm
CANN-robotCANN-robot成员
7月27日 解决了最后一个问题
CANN-robotCANN-robot成员
7月27日 关闭了关联的issue
CANN-robotCANN-robot成员
7月27日 合入了pull request
CANN-robot
CANN-robot成员
7月27日 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike