已合并
补齐算子less、lgamma、digamma ut用例 满足覆盖率要求 #1399
qiang_zq创建于 2月28日
补齐算子less、lgamma、digamma ut用例 满足覆盖率要求 #1399
已合并
qiang_zq创建于 2月28日
qiang_zq
qiang_zq
2月28日

描述

算子less、lgamma、digamma ut用例覆盖率 未达到80%

关联的Issue

https://gitcode.com/cann/ops-math/issues/819

测试

bash build.sh -u --opapi --ops=lgamma,digamma --cov
image.png
image.png

bash build.sh -u --opkernel_aicpu --ops=less --cov
image.png

文档更新

NA

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 qiang_zq 的贡献)
qiang_zqqiang_zq
2月28日 创建了 pull request,commit 8c8cde74
CANN-robot
CANN-robot成员
2月28日 评论:

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
math/less songkai111, zhou-qilong (2/2) zhou-qilong, songkai111 (2/1)
repo-cann/ops-math songkai111, zhou-qilong (2/2) songkai111, zhou-qilong (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robotCANN-robot成员
2月28日 将FelixTang7,zhou-qilong,wangrui_,rxtfeng,loov1,songkai111,zhangyuxiang0119,llimwang,gubaocheng设为评审人
CANN-robotCANN-robot成员
2月28日 将FelixTang7,zhou-qilong,wangrui_,rxtfeng,loov1,songkai111,zhangyuxiang0119,llimwang,gubaocheng设为审查人
CANN-robotCANN-robot成员
2月28日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
2月28日 评论:

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
2月28日 评论:

🔵 source code change are detected, tasks labels is removed in this pull request!

likedislike
CANN-robot
CANN-robot成员
2月28日 评论:

问题/功能描述

本次PR主要包含三项变更:1) 调整了digamma和lgamma两个数学算子的CMake构建脚本,将测试目录重新纳入构建范围;2) 更新了digamma和lgamma算子的单元测试代码,调整了头文件引用路径,并注释了部分耗时较长的精度测试代码以提升测试效率;3) 为less算子新增了两个针对int32数据类型的大规模并行计算场景的单元测试用例。

修改方案描述

  1. 构建脚本调整:移除了math/digammamath/lgamma目录下CMakeLists.txt中排除tests子目录的指令,确保测试代码能被正常构建。
  2. 测试代码优化:更新了digamma和lgamma测试的头文件路径。在多个数据格式(NCHW, NHWC, HWCN等)的测试用例中,注释掉了ut.TestPrecision()调用,改为仅测试工作空间大小获取。同时,将digamma和lgamma测试中针对NC1HWC0格式的预期错误码从ACLNN_ERR_PARAM_INVALID改为ACL_SUCCESS
  3. 新增测试用例:为less算子新增了两个int32数据类型的并行计算测试用例,分别验证了广播计算(shape: [64,32] 与 [32])和相同shape([64,128])大规模数据下的正确性。
likedislike
qiang_zq
qiang_zq
2月28日 评论:

/compile

likedislike
CANN-robotCANN-robot成员
2月28日 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
2月28日 评论:

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
SCA ✅ SUCCESS >>>>>
anti_virus ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_A5 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_A5 ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
UT_Test ✅ SUCCESS
UT_Test_experimental ✅ SUCCESS
Smoke_A900_npupool ✅ SUCCESS >>>>>
harmony-infer ✅ SUCCESS

[2026-02-28 16:18:19]    CI执行结束

likedislike
qiang_zqqiang_zq
2月28日 修改了pull request 的描述
qiang_zqqiang_zq
2月28日 修改了pull request 的描述
qiang_zqqiang_zq
2月28日 修改了pull request 的描述
CANN-robotCANN-robot成员
2月28日 添加了label:api-check-pass
CANN-robotCANN-robot成员
2月28日 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
2月28日 添加了label:ci-pipeline-passed
songkai111成员
2月28日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
2月28日 添加了label:approved
sunday成员
2月28日 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
2月28日 添加了label:lgtm
CANN-robot
CANN-robot成员
2月28日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: songkai111, zhou-qilong.
Reviewers who wrote a comment of /lgtm are: songkai111, zhou-qilong.

likedislike
CANN-robotCANN-robot成员
2月28日 关闭了关联的issue
CANN-robotCANN-robot成员
2月28日 合入了pull request