已合并
新增 apply_ftrl 算子tiling ut #2015
cheng-ziyang2创建于 2月24日
新增 apply_ftrl 算子tiling ut #2015
已合并
cheng-ziyang2创建于 2月24日
cheng-ziyang2成员
2月24日

描述

问题/功能描述
本次PR旨在恢复ApplyFtrl算子针对FLOAT32、FLOAT16和BFLOAT16三种数据类型的单元测试,以验证其分片(tiling)逻辑的正确性。这些测试用例先前被注释而无法执行,导致相关功能的测试覆盖不足。通过启用这些测试,可以增强代码质量保障,确保算子在多种数据类型和输入形状下的分片计算符合预期。

修改方案描述
具体修改是将测试文件中三个被注释的测试函数(apply_ftrl_testcase_float32、apply_ftrl_testcase_float16 和 apply_ftrl_testcase_bfloat16)恢复为可执行状态。同时,更新了每个测试用例中用于验证的预期结果字符串(expectStr),使其与当前实际的分片算法输出保持一致,从而确保测试能够有效运行并通过。

关联的Issue

关联Issue #1121

测试

ut测试通过

文档更新

不涉及

类型标签

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 cheng-ziyang2 的贡献)
Ccheng-ziyang2成员
2月24日 创建了 pull request,commit c455def6
CANN-robot
CANN-robot成员
2月24日 评论:

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
optim chenqi317, fanqirui (2/2) chenqi317 (1/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月24日 将crystalhu,yu-xinjie62,fanqirui,zhou-qilong,chenqi317,liubo75,tangweiwei2,wangyongguang,zhang-wu设为评审人
CANN-robotCANN-robot成员
2月24日 将crystalhu,yu-xinjie62,zhou-qilong,chenqi317,liubo75,tangweiwei2,wangyongguang,zhang-wu设为审查人
CANN-robotCANN-robot成员
2月24日 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
2月24日 评论:

CLA Signature Pass

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

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

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

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

问题/功能描述

本次PR旨在激活ApplyFtrl算子针对FLOAT32、FLOAT16和BFLOAT16三种数据类型的单元测试,以验证其分片(tiling)逻辑的正确性。这些测试用例先前被注释而无法执行,导致相关功能的测试覆盖不足。通过启用这些测试,可以增强代码质量保障,确保算子在多种数据类型和输入形状下的分片计算符合预期。

修改方案描述

具体修改是将测试文件中三个被注释的测试函数(apply_ftrl_testcase_float32apply_ftrl_testcase_float16apply_ftrl_testcase_bfloat16)恢复为可执行状态。同时,更新了每个测试用例中用于验证的预期结果字符串(expectStr),使其与当前实际的分片算法输出保持一致,从而确保测试能够有效运行并通过。

likedislike
cheng-ziyang2成员
2月24日 评论:

/compile

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

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

任务名称状态日志下载链接
codecheck ✅ SUCCESS >>>>>
anti_virus ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
Compile_Ascend_X86 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_mobile_station ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_single ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_experimental ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_X86_950 ✅ SUCCESS >>>>> >>>>>
Compile_Ascend_ARM_950 ✅ SUCCESS >>>>> >>>>>
API_Check ✅ SUCCESS >>>>>
Pre_Compile ✅ SUCCESS
pre_comment ✅ SUCCESS >>>>>
UT_Test_ophost ✅ SUCCESS
UT_Test_opapi ✅ SUCCESS
UT_Test_kernel ✅ SUCCESS
UT_Test_opgraph ✅ SUCCESS
Smoke_A900_npupool ✅ SUCCESS >>>>>

[2026-02-24 19:40:20]    CI执行结束

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

/approve

likedislike
CANN-robotCANN-robot成员
2月25日 添加了label:approved
范其瑞
范其瑞成员
2月26日 评论:

/lgtm

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

Review Guide

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

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