已合并
update: 更新文件 02.04_introduction_to_kernel_functions_based_on_add_operator.ipynb #555
update: 更新文件 02.04_introduction_to_kernel_functions_based_on_add_operator.ipynb #555
已合并
chenmingyu创建于 20 天前
chenmingyu
chenmingyu成员
20 天前

描述

关联的Issue

测试

文档更新

类型标签

其他信息

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 chenmingyu 的贡献)
chenmingyuchenmingyu成员
20 天前 创建了 pull request,commit 4e6c1246
atomgit-bot
atomgit-bot
20 天前 评论:

变更摘要

该 PR 对教程笔记本 02.04_introduction_to_kernel_functions_based_on_add_operator.ipynb 进行了小幅清理,删除了两处代码单元格中多余的 return 0; 语句。这些语句位于精度验证逻辑的末尾,移除后不影响函数行为,仅使示例代码更加简洁。

主要改动

  • 移除第一处冗余 return 0;:在第一个代码示例的精度验证失败分支 (return 1;) 之后,删除了不再需要的 return 0; 语句。
  • 移除第二处冗余 return 0;:在第二个代码示例中,同样在精度验证失败分支之后、main 函数定义之前,删除了多余的 return 0; 语句。
likedislike
atomgit-bot
atomgit-bot
20 天前 评论:

代码审查

✅ 未发现问题

likedislike
chenmingyuchenmingyu成员
20 天前 关联了issue:[Course-Feedback|课程反馈]: 02.04 VerifyResult函数存在不可达死代码
CANN-robotCANN-robot成员
20 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
20 天前 评论:

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-cann/cann-learning-hub waimaidaole, fuyangchenghu (2/2) waimaidaole (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robot
CANN-robot成员
20 天前 评论:

CLA Signature Pass

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

likedislike
meme成员
20 天前 评论:

/approve
/lgtm

likedislike
CANN-robotCANN-robot成员
20 天前 添加了label:approved
fuyangchenghu成员
20 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
20 天前 添加了label:lgtm
CANN-robotCANN-robot成员
20 天前 合入了pull request