已开启
docs(svd_quant): 修正路径变量多余美元符 #255
不喜欢雨天ༀ创建于 23 天前
docs(svd_quant): 修正路径变量多余美元符 #255
已开启
不喜欢雨天ༀ创建于 23 天前
不喜欢雨天ༀ
23 天前

描述

修复 svd_quant 中英文指南 8 处带多余美元符的 cd 命令。

  • 将 cd pathpath/amct_ops 改为 cd "$path/amct_ops"。
  • 将 cd pathpath/tests/amct_ops 改为 cd "$path/tests/amct_ops"。
  • 同步修改 README.md 与 README_en.md,保持双语一致。
  • 仅修正 shell 路径拼接,不改动构建参数或算子代码。

关联的Issue

#176

测试

  • 全目录搜索确认 README.md / README_en.md 均无 pathpath/ 残留。
  • 两份文档各有 4 条修正后的带引号 cd 命令。
  • 变量展开后的 amct/amct_ops、amct/tests/amct_ops 均存在。
  • 对应 ops_build.sh、test_svd_quant.py 均存在。
  • git diff --check:通过。

文档更新

同步更新 svd_quant 中英文编译、安装与测试命令。

类型标签

likedislike
合并受阻
不喜欢雨天ༀ不喜欢雨天ༀ
23 天前 创建了 pull request,commit 340c1d5d
不喜欢雨天ༀ不喜欢雨天ༀ
23 天前 关联了issue:[Docs] svd_quant 中英文指南的 cd $path$/... 含多余美元符,编译/安装/测试命令均进入错误目录
atomgit-bot
atomgit-bot
23 天前 评论:

变更摘要

本 PR 针对 svd_quant 的中英文指南文档进行路径变量写法修复:将带多余美元符的 cd $path$/amct_opscd $path$/tests/amct_ops 统一修正为带引号的 cd "$path/amct_ops"cd "$path/tests/amct_ops",并同步更新 README.mdREADME_en.md,保持双语一致。该变更仅修正 shell 路径拼接,不改动构建参数或算子代码。

主要改动

  • 修正 README.md 中的 cd 命令: 将文档中 4 处 $path$/... 形式的路径变量拼接改为 "$path/..." 带引号写法,消除多余美元符,确保路径展开正确。
  • 修正 README_en.md 中的 cd 命令: 同步将英文文档中 4 处相同问题修正为 cd "$path/amct_ops"cd "$path/tests/amct_ops",保持与中文文档一致。
  • 保持双语内容同步: 两份文档均无 $path$/ 残留,修正后的命令变量展开路径与 amct/amct_opsamct/tests/amct_ops 实际目录一致。
likedislike
atomgit-bot
atomgit-bot
23 天前 评论:

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

likedislike
CANN-robotCANN-robot成员
23 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
23 天前 评论:

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.
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/amct ❌ (0/2)(You can also ask: wulinyu, 姚广修, sophia1213, 金瑛琪, zhuliangying) ❌ (0/1)(You can also ask: wulinyu, zhuliangying, li_ting, 袁江, 姚广修)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
23 天前 将wulinyu,zhuliangying,yaoguangxiu,lianghengyi,fujun19,sophia1213,li_ting,jinyingqi,yuanjiang_设为评审人
CANN-robotCANN-robot成员
23 天前 将wulinyu,zhuliangying,yaoguangxiu,lianghengyi,fujun19,sophia1213,li_ting,jinyingqi,yuanjiang_设为审查人
yaoguangxiu成员
22 天前 评论:

compile

likedislike
不喜欢雨天ༀ不喜欢雨天ༀ
22 天前 update merge request[project id: 8782573, iid: 255, commit_id: 94439cdb61a292035569f4dc87f16d1a5a5db506] virtual merging success
CANN-robotCANN-robot成员
22 天前 添加了label:ci-pipeline-running
CANN-robotCANN-robot成员
22 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
22 天前 添加了label:ci-pipeline-failed
不喜欢雨天ༀ
不喜欢雨天ༀ
18 天前 评论:

/compile

likedislike
不喜欢雨天ༀ不喜欢雨天ༀ
18 天前 update merge request[project id: 8782573, iid: 255, commit_id: 1584e438154bd4c68cd0c12ef2d49f846c169fdc] virtual merging success
此处折叠了8条事件消息 查看更多
CANN-robotCANN-robot成员
18 天前 添加了label:ci-pipeline-failed