Merged
docs: 修复注释与文档不一致(batch_to_space_nd、broadcast_to、chunk_cat 文件注释) #6596
docs: 修复注释与文档不一致(batch_to_space_nd、broadcast_to、chunk_cat 文件注释) #6596
Merged
TangPCAdd on 19 days ago
TangPC
TangPCCollaborators
19 days ago

描述

修正以下文件的注释与实际代码不一致问题,仅修改注释,不涉及业务逻辑、接口行为或配置默认值。

  • conversion/batch_to_space_nd/op_kernel/arch35/batch_to_space_nd_large_c.h:\file 注释由 batch_to_space_largec.h 修正为 batch_to_space_nd_large_c.h。
  • conversion/batch_to_space_nd/op_kernel/arch35/batch_to_space_nd_small_c.h:\file 注释由 batch_to_space_smallc.h 修正为 batch_to_space_nd_small_c.h。
  • conversion/broadcast_to/examples/test_geir_broadcast_to.cpp:\file 注释由 test_geir_add.cpp 修正为 test_geir_broadcast_to.cpp。
  • conversion/chunk_cat/op_api/aclnn_chunk_cat.cpp:\file 注释由 aclnn_cat.cpp 修正为 aclnn_chunk_cat.cpp。

关联的Issue

测试

文档更新

无独立文档文件更新;本次仅修正上述源文件的注释。

类型标签

待确认项

无。

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

变更摘要

本 PR 属于文档注释修正,仅更新四个源文件中 Doxygen 的 \file 注释,使其与实际文件名一致,不涉及业务逻辑、接口行为或配置默认值的变化。\brief 描述行保持原样。

主要改动

  • batch_to_space_nd 大核文件注释修正:batch_to_space_nd_large_c.h 的 \file 由 batch_to_space_largec.h 更正为 batch_to_space_nd_large_c.h,解决注释与文件名不符的问题。
  • batch_to_space_nd 小核文件注释修正:batch_to_space_nd_small_c.h 的 \file 由 batch_to_space_smallc.h 更正为 batch_to_space_nd_small_c.h。
  • broadcast_to 示例文件注释修正:test_geir_broadcast_to.cpp 的 \file 由 test_geir_add.cpp 更正为 test_geir_broadcast_to.cpp,消除从其他示例复制残留的文件名。
  • chunk_cat 接口文件注释修正:aclnn_chunk_cat.cpp 的 \file 由 aclnn_cat.cpp 更正为 aclnn_chunk_cat.cpp,使文件头注释与 aclnn_chunk_cat.h 的引用对象一致。

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

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

代码审查

✅ 未发现问题


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

likedislike
Incorrect?
CANN-robotCANN-robotCollaborators
19 days ago Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
19 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
conversion/broadcast_to ✅ 王林木, 张磊 (2/2) ✅ 王林木, 张磊 (2/1)
repo-cann/ops-math ✅ 王林木, 张磊 (2/2) ✅ 王林木, 张磊 (2/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
CANN-robot
CANN-robotCollaborators
19 days ago Comments:

CLA Signature Pass

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

likedislike
CANN-robotCANN-robotCollaborators
19 days ago Set FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,llimwang,gubaocheng,andong_hw,xuejinghui,onanfield,zhanw_coding,clinglai0517 as approver
CANN-robotCANN-robotCollaborators
19 days ago Set FelixTang7,yue-ma,zhou-qilong,wangrui_,rxtfeng,loov1,zhangzijie,jia0liang,zl_hw,songkai111,chenxingyu18,llimwang,gubaocheng,andong_hw,xuejinghui,onanfield,zhanw_coding,clinglai0517 as approver
TangPC
TangPCCollaborators
19 days ago Comments:

compile

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

流水线分析报告 — ops-math run#5534

流水线: https://gitcode.com/CANN/ops-math/actions/runs/6b271d5dfd6e4d27a097dd287462799b | PR#6596 | 2026-10-22 15:58:15 ~ 2026-10-22 16:10:05 | 失败 Job 1 个

定界结论: 环境问题 / Runner机器异常 — TypeError: Cannot read properties of und
  • 失败类型: 环境问题(非 PR 代码导致)
  • 根因: CI 组件 JS 空引用崩溃(字段: bufMD5, 平台/组件问题, 可重试, 持续失败报维护者)
  • 处置建议: ① 直接重新触发流水线(平台组件瞬时异常, 大概率恢复)
    ② 重试仍持续失败 → 报 CI 平台维护者处理 obs-download 组件缺陷
  • 排除项: 失败发生在 download 步骤的 obs-download 平台组件内部(早于 pre-commit 检查执行), 与 PR#6596 变更代码无关

失败根因分析

根因 #1: 环境问题 / Runner机器异常 — TypeError: Cannot read properties of und
  • 判据: CI 组件 JS 空引用崩溃(字段: bufMD5, 平台/组件问题, 可重试, 持续失败报维护者)
  • 定位步骤: 2_download.log
  • 影响 Job: CodeCheck_precommit
  • 根因证据(日志原文):
[2026-10-22 15:59:17.411+08:00] /runner/workers/***.version/_actions/obs-download/1.1.7/dist/index.js:1
[2026-10-22 15:59:17.422+08:00] TypeError: Cannot read properties of undefined (reading 'bufMD5')
[2026-10-22 15:59:17.423+08:00]     at calculateDownloadCheckpointMD5 (/runner/workers/***.version/_actions/obs-download/1.1.7/dist/index.js:1:112915)
[2026-10-22 15:59:17.424+08:00]     at currStepEnd (/runner/workers/***.version/_actions/obs-download/1.1.7/dist/index.js:1:115249)
结构化定界摘要
[{"repo": "ops-math", "run_number": 5534, "Failed_Type": "环境问题",
  "Failed_Sub_Type": "Runner机器异常",
  "Failed_Reason": "CI 组件 obs-download 1.1.7 在 download 步骤 JS 空引用崩溃(TypeError: Cannot read properties of undefined (reading 'bufMD5')), 平台组件问题, 与 PR 代码无关",
  "evidence": "[2026-10-22 15:59:17.422+08:00] TypeError: Cannot read properties of undefined (reading 'bufMD5')",
  "suggestion": "重新触发流水线验证; 持续失败则报 CI 平台维护者处理 obs-download 组件缺陷",
  "jobs": "CodeCheck_precommit"}]

由门禁失败定界分析自动生成 | repo: ops-math | run: 6b271d5dfd6e4d27a097dd287462799b | PR#6596

likedislike
TangPC
TangPCCollaborators
19 days ago Comments:

compile

likedislike
CANN-robot
CANN-robotCollaborators
19 days ago Comments:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robotCollaborators
19 days ago Removed label: ci-pipeline-failed
CANN-robotCANN-robotCollaborators
19 days ago Added label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
19 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
19 days ago Added label: ci-pipeline-passed
llimwangCollaborators
19 days ago Comments:

/lgtm
/approve

likedislike
CANN-robotCANN-robotCollaborators
19 days ago Added label: approved
zl_hwCollaborators
18 days ago Comments:

/lgtm
/approve

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