已合并
【NSAG】UT补充 #5703
【NSAG】UT补充 #5703
已合并
xuchengyang创建于 5月21日
xuchengyang
5月21日

描述

关联的Issue

测试

文档更新

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 xuchengyang 的贡献)
Xxuchengyang
5月21日 创建了 pull request,commit 577c75d6
CANN-robot
CANN-robot成员
5月21日 评论:

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
attention 於欣洁, cc-z (2/2) cc-z (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
Xxuchengyang
5月21日 修改了pull request 的描述
CANN-robotCANN-robot成员
5月21日 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
5月21日 将juyangokok,monologue815,crystalhu,yu-xinjie62,huangwei791,jiang-lirui,yang-binrong,Allan_Yu,liudan12,wangzhe123456789,cc-z,shen_weiling,gaoxiang618,songjionghui,wenxin_fight,miaofangzheng,mabing1118,tang-hao-hw-gitcode,haijie_699874,zzy__设为评审人
CANN-robotCANN-robot成员
5月21日 将juyangokok,monologue815,crystalhu,yu-xinjie62,huangwei791,jiang-lirui,yang-binrong,Allan_Yu,liudan12,wangzhe123456789,cc-z,shen_weiling,gaoxiang618,songjionghui,wenxin_fight,miaofangzheng,mabing1118,tang-hao-hw-gitcode,haijie_699874,zzy__设为审查人
xuchengyang
5月21日 评论:

compile

likedislike
Xxuchengyang
5月21日 update merge request[project id: 7673863, iid: 5703, commit_id: 7fcf484fe0e1e654cdf74a807813f33ed1c19355] virtual merging success
CANN-robotCANN-robot成员
5月21日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
5月21日 评论:

流水线任务触发成功
任务链接 [d17f28e1363b4020bba7390b317efa66][流水线指导]

任务名称状态日志下载链接
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Pre ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
Compile_Ascend_X86_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_mobile_station_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_X86_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_experimental_ubuntu24 ✅ SUCCESS >>>>>
Compile_X86_monitor_910b ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_910c ✅ SUCCESS >>>>> >>>>>
Compile_X86_monitor_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental_950_ubuntu24 ✅ SUCCESS >>>>>
Compile_Ascend_ARM_experimental_950_ubuntu24 ✅ SUCCESS >>>>>
UT_Test ✅ SUCCESS
PreSmoke_A900 ✅ SUCCESS >>>>>
API_Check ✅ SUCCESS >>>>>
PreSmoke_ATK_Test_A2 ✅ SUCCESS >>>>>

[2026-05-21 17:39:11]    CI执行结束

likedislike
CANN-robot
CANN-robot成员
5月21日 评论:

流水线任务触发成功
任务链接 [16385b0cdec8492c99b7e1892ee53235][流水线指导]

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
codecheck_style ✅ SUCCESS codecheck sync failed
StaticCheck_codespell_check ✅ SUCCESS
StaticCheck_link_validity_check ✅ SUCCESS
StaticCheck_resource_existence_check ✅ SUCCESS
StaticCheck_tag_closed_check ✅ SUCCESS
StaticCheck_markdownlint ✅ SUCCESS
codecheck_checkpr ✅ SUCCESS
codecheck_precommit ✅ SUCCESS >>>>>

[2026-05-21 17:18:32]    CI执行结束

likedislike
atomgit-bot
atomgit-bot
5月21日 评论:

代码审查

本次变更是 NSA 梯度算子测试框架从旧的自定义 Case 体系迁移到 gtest 体系,新增了 op_api、tiling、infershape 三类 UT 并补充了负向用例。主要风险在于部分旧正向场景(D2=192 等 head dim、deterministic 模式)在新 UT 中未覆盖。

类型 数量
🔴 阻塞 0 处
🟡 建议 1 处

💬 仅评论

likedislike
atomgit-bot
atomgit-bot5月21日进行代码检视1
attention/nsa_selected_attention_grad/tests/ut/op_host/arch22/test_nsa_selected_attention_grad_tiling.cpp
@@ -0,0 +313,4 @@
313+ 8192);
314+ 
315+ ExecuteTestCase(para, ge::GRAPH_FAILED);
316+}
atomgit-bot
atomgit-bot5月21日评论:

🟡 Medium Priority

被删除的旧测试 ts_nsag_tc.cpp 包含 2 个 D2=192 的正向用例(case_nsa_selcted_attention_grad_0 / _1),新 tiling UT 中所有正向用例均使用 D2=128。若 D2=192 路径存在特殊 tiling 逻辑,将失去测试保护。

likedislike
CANN-robotCANN-robot成员
5月21日 添加了label:api-check-pass
CANN-robotCANN-robot成员
5月21日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
5月21日 添加了label:ci-pipeline-passed
於欣洁成员
5月23日 评论:

/lgtm

likedislike
赵臣臣成员
5月23日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
5月23日 添加了label:lgtmapproved
CANN-robot
CANN-robot成员
5月23日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: cc-z.
Reviewers who wrote a comment of /lgtm are: cc-z, 於欣洁.

likedislike
CANN-robot
CANN-robot成员
5月23日 评论:

The MR can not be merged, because of CodeReview discussion not resolved

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

likedislike
Xxuchengyang
5月23日 解决了最后一个问题
CANN-robot
CANN-robot成员
5月23日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: cc-z.
Reviewers who wrote a comment of /lgtm are: cc-z, 於欣洁.

likedislike
CANN-robotCANN-robot成员
5月23日 合入了pull request