已合并
docs(triton): 修正评测说明中的固定仓库路径 #31
docs(triton): 修正评测说明中的固定仓库路径 #31
已合并
bububu创建于 29 天前
bububu
29 天前

描述

issue #4 已复现:SKILLS/triton-npu-optimize/references/eval.md 的评测准备步骤固定切换到 /root/work/workspace/mcg/oil-gas-ops-prospect,在其他挂载路径下会立即失败,后续相对路径命令无法从仓库根目录运行。

本 PR 只改这一份评测文档,逐处对应 diff:

  • 开头工作目录说明改为要求从本容器实际挂载的仓库根目录启动 shell,并明确不要假定固定路径。
  • 删除环境变量代码块中的固定绝对 cd 行。其余环境设置、测试、性能和编译步骤不变,不涉及算子或运行时行为。

关联的Issue

关联 issue #4

测试

修复前复现(真实输出):

bash: line 1: cd: /root/work/workspace/mcg/oil-gas-ops-prospect: No such file or directory
command=cd /root/work/workspace/mcg/oil-gas-ops-prospect
exit_code=1

修复后静态回归检查:

python3 - <<'PY'
from pathlib import Path
doc = Path("SKILLS/triton-npu-optimize/references/eval.md").read_text(encoding="utf-8")
assert "/root/work/workspace/mcg/oil-gas-ops-prospect" not in doc
assert "实际挂载的仓库根目录" in doc
assert Path("tests/triton/real_fft/test_triton_fft_real.py").is_file()
assert Path("triton-ascend/tools/aot_compile_check.py").is_file()
print("PASS: 已移除固定仓库路径;仓库根目录说明及测试/编译相对路径有效")
PY

实际输出:PASS: 已移除固定仓库路径;仓库根目录说明及测试/编译相对路径有效。本 PR 仅改文档,未运行 pytest、编译或 NPU 测试。

文档更新

已更新 SKILLS/triton-npu-optimize/references/eval.md 的工作目录约定并移除环境相关的固定仓库路径。

类型标签

当前PR是否有AI参与:

[] 否
[x] 是
__1. Agent平台信息: Codex CLI 0.153.4
__2. 模型信息: gpt-5.6-luna
__3. Prompt摘要: 在已有 issue #4 基础上独立复现固定仓库路径缺陷,仅修改评测文档工作目录说明,逐 hunk 对账、执行静态回归并按 PR-first 流程提交和回读。

希望检视人员了解:

  1. 人工参与说明:用户指定已有 issue、case、fork 和 PR-first 流程;Codex 独立核对源文件行号、复现问题、实施文档修复、逐 hunk 对账、执行静态回归及服务端回读。尚无第二位人工 reviewer 复核。
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 bububu 的贡献)
Bbububu
29 天前 创建了 pull request,commit 40947ffa
Bbububu
29 天前 关联了issue:[Documentation|文档反馈]: Triton NPU 评测说明固定仓库路径导致其他挂载环境无法执行
CANN-robotCANN-robot成员
29 天前 添加了label:ai-co-authored
CANN-robotCANN-robot成员
29 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
29 天前 评论:

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
sig-oil-gas-engineering ✅ 胡艺馨, 徐澜 (2/2) ✅ 胡艺馨 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
bububu
29 天前 评论:

compile

likedislike
Bbububu
29 天前 预合并成功(commit_id: 7fb8635b6b6557e789fb8ba3c4e0c39d4eac20c8)
CANN-robot
CANN-robot成员
29 天前 评论:
🚀 CI 流水线已启动
📋 执行详情: 点击查看流水线
likedislike
CANN-robotCANN-robot成员
29 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
29 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
29 天前 添加了label:ci-pipeline-passed
newbgirl
newbgirl成员
27 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
27 天前 添加了label:approved
Chris-Lan成员
27 天前 评论:

/lgtm

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