Closed
[L2] fix: 5 处文档图片相对链接少了一层目录(../figures → en/figures),目标文件存在但链接 404 #132
eieieiAdd on Oct 2Closed at 16 days ago
[L2] fix: 5 处文档图片相对链接少了一层目录(../figures → en/figures),目标文件存在但链接 404 #132
Closed
eieieiAdd on Oct 2Closed at 16 days ago
eieiei
eieiei
Oct 2

背景

docs/quant_mode_introduction.md 位于 docs/ 下,里面 5 处图片写成 ../figures/xxx.png,按相对路径解析到仓库根的 figures/——该目录在仓内不存在;图片实际在 docs/en/figures/ 下。⇒ 5 处链接全 404,页面图片不显示。

修改(5 行)

../figures/ → en/figures/,只动 docs/quant_mode_introduction.md 的 5 行。

验证(本机静态,可复核)

  • 用 git ls-files 建立跟踪文件集合,改前 5 条相对链接解析结果均为 figures/*.png(不在集合内 ⇒ 404),改后均解析到 docs/en/figures/*.png(在集合内 ⇒ 可达)。断言写在提交流水里:改后该文件内所有非 http、非锚点的相对链接都能解析到跟踪文件,无一走空。
  • 该文件是 LF(无 CRLF),改后仍是 LF;末字节为换行、无行尾空白。
  • 另两份同名副本(docs/en/context/、docs/zh/context/)的链接实测均可解析,不需同改;docs/zh/figures/ 下是中文文件名图片,与本处英文图名不同一套。

与上一版 head 的差别(本次在最新 master 上重做,原 head 与 master 冲突后已 abort)

原 PR 声称 6 处,重核后收窄为 5 处:

  • docs/zh/develop/aicpu_develop_guide.md 的 Kernel UT 链接已由他人改对(master 现在指向存在的 examples/add_example_aicpu/op_kernel_aicpu/add_example_aicpu.cpp)⇒ 不再是缺陷;
  • 1 处是指向目录的链接(../../../examples/add_example_aicpu),我原先的校验只查文件所以误判为 404。

想请模块内的评审同学看一下路径口径:本仓 docs/ 根下的文档引用 docs/en/figures/ 是否有约定要改指 zh/figures/(若希望按语言分套,我可以按评审意见调)。

关联 Issue

  • cann/ops-ras #63:本 PR 修的就是它描述的 5 处破图(docs/quant_mode_introduction.md 的 ../figures/*.png 解析到仓库根的 figures/,实际在 docs/en/figures/)。
  • cann/ops-ras #59:同一文件的另一面——这份根级副本与 docs/en/context/quant_mode_introduction.md 逐字节相同(md5 5fb94cc6ab8a),所以「修链接」与「删孤儿副本、只留 en/context 那份」是两条路。

本 PR 走的是最小改动(只把 5 条相对路径改对,+5/-5,不动去留)。若模块判定应删除根级副本,我这边撤这条 PR、按 #59 的口径另开一条删副本的改动,不两边同时留。

codecheck_Pr 自 2026-10-02 起 8 次全红(含我只改一行、以及本仓其它 PR 同期通过的情况),且明细页需登录、提交者侧看不到。若卡点正是「PR 必须关联 Issue」,上面的关联已补上,麻烦再跑一次;如果是别的规则,恳请告知具体条目名,我按条目修。

likedislike
The current Pull Request has been closed by eieiei.
eieieieieiei
Oct 2 Created pull request, commit 16986557
atomgit-bot
atomgit-bot
Oct 2 Comments:

变更摘要

本 PR 修复 markdown 文档中 6 处相对链接路径缺少中间目录层导致的断链问题:这些链接解析后的目标文件在仓库中不存在(渲染与点击均 404),但对应目标文件实际存在于仓库的其他目录中。改动仅替换链接的目标路径,使其相对链接所在文件目录重新解析后指向真实存在的文件,链接文字、锚点及其余内容保持不变。共涉及 docs/quant_mode_introduction.md 与 docs/zh/develop/aicpu_develop_guide.md 两个文件,增删行数均为 6。

主要改动

  • 修正量化模式文档的图片链接路径: docs/quant_mode_introduction.md 中 5 处 ../figures/*.png 链接(pertensor_quantization.png、perchannel_quantization.png、pertoken_quantization.png、pergroup_quantization.png、perblock_quantization.png)统一改为 en/figures/*.png,补上缺失的 en 目录层。
  • 修正 aicpu 开发指南的示例代码链接路径: docs/zh/develop/aicpu_develop_guide.md 中的链接由 ../../../examples/add_example_aicpu/tests/ut/op_kernel_aicpu/test_add_example.cpp 改为 ../../../examples/add_example/tests/ut/op_kernel/test_add_example.cpp。
  • 保持链接语义不变: 仅修改链接目标路径,链接文字与锚点未改动,因此不影响文档其他内容与阅读顺序。
likedislike
Incorrect?
atomgit-bot
atomgit-bot
Oct 2 Comments:

🤖 本次改动均为数据 / 生成 / 二进制文件(如测试用例、锁文件、媒体资源),没有可审查的代码逻辑,已跳过 AI 代码审查。

likedislike
Incorrect?
CANN-robotCANN-robotCollaborators
Oct 2 Added label: cann-cla/yes
CANN-robot
CANN-robotCollaborators
Oct 2 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

⚠️ This PR does not yet meet the following requirements:lgtm (requires ≥ 2 person(s) per module)、approve (requires ≥ 1 person(s) per module)

Module Approval Details

module lgtm status approve status
repo-cann/ops-ras ❌ (0/2)(You can also ask: 唐燕峰, 周奇龙, 徐鲁威, 陈兴宇, 柳宗谷) ❌ (0/1)(You can also ask: 柳宗谷, 唐燕峰, 陈兴宇, 周奇龙, 徐鲁威)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
111 messages are collapsed here。 See more
CANN-robotCANN-robotCollaborators
16 days ago Added label: api-check-pass
CANN-robotCANN-robotCollaborators
16 days ago Removed label: ci-pipeline-running
CANN-robotCANN-robotCollaborators
16 days ago Added label: ci-pipeline-failed
eieiei
eieiei
16 days ago Comments:

【本 PR 自行关闭 · 已被 !139 取代】

先说结论:cann/ops-ras !139(@NUAA_wqy)对同一个文件 docs/quant_mode_introduction.md 的处理是整份删除(0 +/59 -,现 labels=['cann-cla/yes','ci-pipeline-passed'],mergeable=True),这正是 Issue #59 里维护者认可的做法。我这 5 行是把孤儿副本里的图片路径改成 en/figures/…,一旦副本被删就没有承载对象了,所以不再占评审位,主动关闭。

  • 若 !139 因故没能合入,请在这条下面说一声,我立刻把本 PR 重新推上:head af9dbe4e7079(单提交,1 文件 +5/-5),不用重写。

下面是这条 PR 挂了 20 天期间测出来的 codecheck_Pr 定界数据,留在这里给后面遇到同样红的人:

  1. 分母(只读各 PR 最新评论里的 cann-robot 报告表格,口径 = GET /repos/cann/ops-ras/pulls?state=all&per_page=100&page=1 取最近 25 个,再对每个 GET /pulls/{n}/comments 取最后一条 cann-robot 报告):codecheck_Pr ✅ SUCCESS 22 个,另外 3 个还没跑出该作业,FAILED 的只有本 PR。其中 !174/!175/!177/!178 是我自己的四条,都过了 ⇒ 与提交账号无关,也与「仓库把这个作业配坏了」无关。
  2. 我原先的猜测是错的,以此条为准:我以为恒红是因为 PR 没关联 Issue,于是把本 PR 关联了我自己早先立的 #59 / #63(GET /pulls/132/issues 读回 ['59', '63']),再发裸 /compile。13:48 的报告:codecheck/SCA/antipoison/codecheck_codestyle 全 ✅、codecheck_precommit ⚠️ WARNING,而 codecheck_Pr 仍 ❌ FAILED(自 10-02 起第 10 次)。关联 Issue 只带来了一个新标签 api-check-pass。
  3. 单变量对照:!139 删这个文件 → codecheck_Pr ✅ SUCCESS;本 PR 改这个文件 → 恒 ❌ FAILED;两条 PR 的 base 都是 master,改的都是同一个路径。⇒ 触发点落在这份孤儿副本被修改这件事上,但具体是哪条规则、规则名叫什么,我从接口看不到,请模块内的评审同学看一下(想请模块内的评审同学看一下:如果你知道 codecheck_Pr 对 docs/ 根级文件的规则,麻烦回一句,我把结论补进 #59,避免下一个人再盲跑十次)。

顺带一条对平台可读性的观测(不是本仓的问题,先记在这儿):报告里 FAILED 那一行的日志链接渲染成 <a href=></a>——空 href,连 SUCCESS 行末尾那个 >>>>> 都没有 ⇒ 作业失败时贡献者拿不到任何日志,只能靠猜。上面这张表是我从 GET /repos/cann/ops-ras/pulls/132/comments 的原始 HTML 里正则抽出来的(<td><strong>(job)</strong></td>\s*<td>(status)</td>\s*<td><a href=([^>]*)>),任何人重跑一次 /compile 都能复现同样的空 href。

likedislike
eieieieieiei
16 days ago Close pull request