Merged
docs: 修复注释与文档不一致(LogicalAnd 与 Pow2 调用说明) #6488
docs: 修复注释与文档不一致(LogicalAnd 与 Pow2 调用说明) #6488
Merged
TangPCAdd on 22 days ago
TangPC
TangPCCollaborators
22 days ago

描述

  • experimental/math/logical_and/README.md:链接文字改为 aclnnLogicalAnd,说明中的算子名 SelectV2 -> LogicalAnd。
  • experimental/math/pow2/README.md:链接文字改为 aclnnPow2,说明中的算子名 SelectV2 -> Pow2。

关联的Issue

测试

  • pre-commit(含 OAT):通过
  • Markdown 链接/图片检查:无新增失效项
  • Markdown 代码围栏检查:通过
  • 全部 26 个分支合并验证:无冲突

文档更新

  • experimental/math/logical_and/README.md
  • experimental/math/pow2/README.md

类型标签

likedislike
Pull Request has been successfully merged by CANN-robot.
(Thank you for TangPC's contribution.)
TangPCTangPCCollaborators
22 days ago Created pull request, commit c7008ca1
atomgit-bot
atomgit-bot
22 days ago Comments:

变更摘要

本 PR 为纯文档修正,目的是使 experimental/math/logical_and/README.md 与 experimental/math/pow2/README.md 中「调用方式」表格的说明文字与实际的接口文档链接、算子名称保持一致。此前两处说明误将链接文字写成了样例文件名、并把算子名统一误标为 SelectV2,本次将其分别更正为对应的算子名。不涉及任何代码、接口或配置的行为变化。

主要改动

  • logical_and 链接文字更正: 将 aclnn调用 行中的链接文字由 test_aclnn_logical_and 改为 aclnnLogicalAnd,指向同一 ./docs/aclnnLogicalAnd.md 文档。
  • logical_and 算子名更正: 将说明中的算子名由 SelectV2 改为 LogicalAnd,使其与 aclnnLogicalAnd 接口对应。
  • pow2 链接文字更正: 将 aclnn调用 行中的链接文字由 test_aclnn_pow2 改为 aclnnPow2。
  • pow2 算子名更正: 将说明中的算子名由 SelectV2 改为 Pow2,使其与 aclnnPow2 接口对应。

🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
atomgit-bot
atomgit-bot
22 days ago Comments:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

likedislike
Incorrect?
CANN-robotCANN-robotCollaborators
22 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
22 days ago Comments:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
experimental ✅ 张磊, 王林木 (2/2) ✅ 王林木, 张磊 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robotCollaborators
22 days ago Set FelixTang7,yu-xinjie62,yue-ma,zhou-qilong,wangrui_,wei-peng-1991,renruhai,su-yueming,zhajianqing123,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,chenqi317,zl_hw,liu-wei,LiuXinxin23,fullt,LiWei79,songkai111,chenxingyu18,demoauguste,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding,clinglai0517 as approver
CANN-robotCANN-robotCollaborators
22 days ago Set FelixTang7,yu-xinjie62,yue-ma,zhou-qilong,wangrui_,wei-peng-1991,renruhai,su-yueming,zhajianqing123,rxtfeng,luowen203_gg123,loov1,zhangzijie,jia0liang,chenqi317,zl_hw,liu-wei,LiuXinxin23,fullt,LiWei79,songkai111,chenxingyu18,demoauguste,llimwang,gubaocheng,andong_hw,xuejinghui,zhanw_coding,clinglai0517 as approver
TangPC
TangPCCollaborators
21 days ago Comments:

compile

likedislike
TangPCTangPCCollaborators
21 days ago Pre merge succeeded (commit_id: 98c4c1cb71d2835303ebca467bc836ad040a5539)
CANN-robot
CANN-robotCollaborators
21 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
21 days ago Added label: ci-pipeline-running
TangPCTangPCCollaborators
21 days ago Modify description of pull request
llimwangCollaborators
21 days ago Comments:

/lgtm
/approve

likedislike
CANN-robotCANN-robotCollaborators
21 days ago Added label: approved
CANN-robotCANN-robotCollaborators
21 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
21 days ago Added label: ci-pipeline-passed
chenjiaoCollaborators
21 days ago Comments:

/approve

likedislike
zl_hwCollaborators
21 days ago Comments:

/lgtm
/approve

likedislike
CANN-robot
CANN-robotCollaborators
21 days ago Comments:

The following users do not have permission to comment /lgtm or /approve on any module in this PR:
/approve: 陈娇

likedislike
CANN-robotCANN-robotCollaborators
21 days ago Added label: lgtm
CANN-robotCANN-robotCollaborators
21 days ago Closed associated issue
CANN-robotCANN-robotCollaborators
21 days ago Merged pull request