已合并
Remove Ascend950 version restriction for ACLNN support in _chunk_cat function #5357
nomiz创建于 7月2日
Remove Ascend950 version restriction for ACLNN support in _chunk_cat function #5357
已合并
nomiz创建于 7月2日
nomiz成员
7月2日

【合入来源】

如有社区issue,请关联issue链接
请勿携带内部流程信息(需求链接、问题单、内部issue等)

https://gitcode.com/Ascend/op-plugin/issues/182

【修改方案】

请描述修改内容的具体实现,涉及哪些组件之间进行交互,可以用1、2、3、...进行罗列
如果是需求或者重构类的PR,需要补充详细设计文档(说明上下游组件关系、时序图、类图、DFX能力等内容)

A5已支持aclnnchunkcat算子,该pr移除了 _chunk_cat 函数中 aclnn 支持的 Ascend950 版本限制条件

【资料变更】

请确认是否涉及资料变更。如涉及,需要在PR中体现,并简要说明修改内容。如不涉及,需填写“不涉及”

【接口变更】

请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及,需要详细说明接口以及对应的变更内容,同时需要在资料中体现。如不涉及,需填写“不涉及”

【功能验证】

说明测试场景,测试方法。如果本次测试方式与常规单元测试不同,请详细说明您的测试步骤
新增/变更内容是否已新增/适配UT测试用例看护,并补充测试自验证截图

image.png

ut验证结果
image.png

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 nomiz 的贡献)
Nnomiz成员
7月2日 创建了 pull request,commit 8f329380
Nnomiz成员
7月2日 关联了issue:[Bug]: Remove Ascend950 version restriction for ACLNN support in _chunk_cat function
ascend-robotascend-robot成员
7月2日 添加了label:ascend-cla/yes
nomiz成员
7月2日 评论:

compile

likedislike
ascend-robot
ascend-robot成员
7月2日 评论:

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


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
repo-Ascend/op-plugin liangsongwei, htchu (2/2) liangsongwei, htchu (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
7月2日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
7月2日 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
7月2日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
7月2日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
7月2日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
7月2日 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
7月2日 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
7月2日 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
7月2日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
7月2日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
7月2日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
7月2日 添加了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
7月2日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
7月2日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
7月2日 评论:
流水线 PR-pipeline_op-plugin#2362 [ commitID:3d61b61d ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_master_ARM >>>
Build_v2_7_1_ARM >>>
Build_v2_9_0_ARM >>>
Build_v2_10_0_ARM >>>
Build_v2_11_0_ARM >>>
Build_v2_12_0_ARM >>>
Build_master_x86 >>>
Build_v2_7_1_x86 >>>
Build_v2_9_0_x86 >>>
Build_v2_10_0_x86 >>>
Build_v2_11_0_x86 >>>
Build_v2_12_0_x86 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
CodeCheck_pre-commit >>>
开源片段检查 SCA >>>
开发者测试 UT_master_ARM_01 >>>
UT_v2_7_1_ARM >>>
UT_v2_9_0_ARM >>>
UT_v2_10_0_ARM >>>
UT_v2_11_0_ARM >>>
UT_v2_12_0_ARM >>>
流水线 PR-pipeline_op-plugin >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
atomgit-bot
atomgit-bot
7月2日 评论:

变更摘要

此 PR 移除了 _chunk_cat_chunk_cat_out 函数中对 ACLNN 支持的 Ascend950 版本限制条件。由于 A5(Ascend950)已支持 aclnnChunkCat 算子,原先的版本判断 c10_npu::GetSocVersion() < c10_npu::SocVersion::Ascend950 不再必要,只需通过 check_aclnn_kernel_available("aclnnChunkCat") 判断 ACLNN 内核是否可用即可。

主要改动

  • 移除 _chunk_cat 中的 Ascend950 版本限制:在 npu_support_aclnn 静态变量的初始化中,删除了 && c10_npu::GetSocVersion() < c10_npu::SocVersion::Ascend950 条件,使得 ACLNN 内核可用性判断仅依赖 check_aclnn_kernel_available("aclnnChunkCat") 的返回值。
  • 移除 _chunk_cat_out 中的 Ascend950 版本限制:与 _chunk_cat 一致,同步删除了相同的版本限制条件,确保两个重载函数的行为对齐。
likedislike
atomgit-bot
atomgit-bot
7月2日 评论:

代码审查

✅ 未发现问题

likedislike
nomiz成员
7月13日 评论:

compile

likedislike
ascend-robotascend-robot成员
7月13日 删除了label:ci-pipeline-passed
ascend-robotascend-robot成员
7月13日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
7月13日 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
7月13日 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
7月13日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
7月13日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
7月13日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
7月13日 添加了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
7月13日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
7月13日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
7月13日 评论:
流水线 PR-pipeline_op-plugin#2584 [ commitID:3d61b61d ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build_master_ARM >>>
Build_v2_7_1_ARM >>>
Build_v2_9_0_ARM >>>
Build_v2_10_0_ARM >>>
Build_v2_11_0_ARM >>>
Build_v2_12_0_ARM >>>
Build_master_x86 >>>
Build_v2_7_1_x86 >>>
Build_v2_9_0_x86 >>>
Build_v2_10_0_x86 >>>
Build_v2_11_0_x86 >>>
Build_v2_12_0_x86 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
CodeCheck_pre-commit >>>
开源片段检查 SCA >>>
开发者测试 UT_master_ARM_01 >>>
UT_v2_7_1_ARM >>>
UT_v2_9_0_ARM >>>
UT_v2_10_0_ARM >>>
UT_v2_11_0_ARM >>>
UT_v2_12_0_ARM >>>
流水线 PR-pipeline_op-plugin >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
梁松伟
梁松伟成员
7月13日 评论:

/approve

likedislike
ascend-robotascend-robot成员
7月13日 添加了label:approved
htchu成员
7月13日 评论:

/approve

likedislike
ascend-robotascend-robot成员
7月13日 添加了label:lgtm
ascend-robotascend-robot成员
7月13日 关闭了关联的issue
ascend-robotascend-robot成员
7月13日 合入了pull request