已合并
fix: correct qwen3.5 fla-npu run package pattern #2891
fix: correct qwen3.5 fla-npu run package pattern #2891
已合并
yeqm创建于 27 天前
yeqm成员
27 天前

What this PR does / why we need it?

Fix a typo in the Qwen3.5 README fla-npu installation command.

The generated fla-npu run package uses the fla-npu-*.run naming pattern, for example build_out/fla-npu-fla_npu_linux-x86_64.run. The original README used fla-npu_*.run, which cannot match the generated run package and causes the installation command to fail.

Does this PR introduce any user-facing change?

Yes. This PR corrects the Qwen3.5 fla-npu installation command in the user documentation.

Documentation path: examples/qwen3_5/README.md

How was this patch tested?

self test

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 yeqm 的贡献)
Yyeqm成员
27 天前 创建了 pull request,commit 0b334aea
atomgit-bot
atomgit-bot
27 天前 评论:

变更摘要

该 PR 修复了 Qwen3.5 示例文档中 fla-npu 运行包安装命令的文件名匹配模式错误。实际生成的运行包遵循 fla-npu-*.run 命名规则(如 fla-npu-fla_npu_linux-x86_64.run),而原文档错误地使用了 fla-npu_*.run,导致通配符无法匹配生成的包文件,安装命令执行失败。此次变更将文档中的命令修正为正确的模式。

主要改动

  • 修正 examples/qwen3_5/README.md 中的运行包匹配模式:将 fla-npu_*.run 更正为 fla-npu-*.run,使通配符与实际生成的包名格式一致,确保安装命令能够正确工作。
likedislike
atomgit-bot
atomgit-bot
27 天前 评论:

代码审查

该 diff 仅对 examples/qwen3_5/README.md 第 106 行做了一处修改:将安装 fla-npu run 包的命令中的 glob 模式从 fla-npu_*.run 修正为 fla-npu-*.run,以匹配实际生成的 run 包文件名格式(如 fla-npu-fla_npu_linux-x86_64.run)。

经审查:

  1. 修改正确性:旧的 fla-npu_*.run 使用下划线,无法匹配实际生成的文件名(使用连字符 fla-npu-)。修正后的模式 fla-npu-*.run 可以正确匹配。这是一个纯粹的文档错误修复。

  2. 无残留旧模式:修改后该文件中不再存在旧的 fla-npu_ 错误模式。

  3. 无安全性/可靠性问题:该命令仅为本地安装指令中的 shell glob 展开,不涉及权限提升、密钥泄露、不安全的网络操作等安全风险。

  4. 无新增不一致性:该修改仅修正当前文件中的一个拼写错误,不会引入与其他未修改代码的新不一致性。


审查总结

优先级 数量
P0 0
P1 0
P2 0
P3 0

已审查文件

  • examples/qwen3_5/README.md:无问题,修改正确。

整体风险判断:该变更为纯文档修复,无风险,可安全合入。

类型 数量
🔴 阻塞 0
🟡 建议 1

💬 仅评论

likedislike
ascend-robotascend-robot成员
27 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
27 天前 评论:

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


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
repo-Ascend/MindSpeed-MM 王海涛, liuxi_, wanglikai1019 (3/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

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

likedislike
yeqm成员
27 天前 评论:

compile

likedislike
ascend-robot
ascend-robot成员
27 天前 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
27 天前 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
27 天前 评论:

ascend docs pipeline is running...

likedislike
ascend-robot
ascend-robot成员
27 天前 评论:

✅ 文档门禁通过!

检查项 检查结果 详情
markdownlint ✅ 已通过 查看详情
link-validity-check ✅ 已通过 查看详情
resource-existence-check ✅ 已通过 查看详情
tag-closed-check ✅ 已通过 查看详情
likedislike
ascend-robotascend-robot成员
27 天前 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
27 天前 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
27 天前 评论:

✅ 文档门禁通过!

检查项 检查结果 详情
markdownlint ✅ 已通过 查看详情
link-validity-check ✅ 已通过 查看详情
resource-existence-check ✅ 已通过 查看详情
tag-closed-check ✅ 已通过 查看详情
likedislike
Yyeqm成员
27 天前 关联了issue:[Doc]: qwen35 README中,bash build_out/fla-npu_*.run有误
犀牛
犀牛成员
27 天前 评论:

/lgtm

likedislike
LKONE
LKONE成员
27 天前 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
27 天前 添加了label:lgtm
atomgit-bot
atomgit-bot27 天前进行代码检视1
examples/qwen3_5/README.md
@@ -104,3 +104,3 @@
104104# 编译算子 run 包,--soc 需指定为当前机器芯片类型 {ascend910b/ascend910_93/ascend950}
105105bash build.sh --soc=ascend910b --pkg --vendor_name=fla_npu
106-bash build_out/fla-npu_*.run
106+bash build_out/fla-npu-*.run
atomgit-bot
atomgit-bot27 天前评论:

🟡 Medium Priority

变更将 examples/qwen3_5/README.md 第 106 行的 fla-npu_*.run 修正为 fla-npu-*.run,但项目中还有两个文件存在完全相同的错误模式,未被本次 PR 覆盖:

  • examples/qwen3_6/README.md 第 97 行:bash build_out/fla-npu_*.run
  • examples/mcore/qwen3_5/README.md 第 100 行:bash build_out/fla-npu_*.run

这两处同样无法匹配生成的 run 包名(如 fla-npu-fla_npu_linux-x86_64.run),用户在按照这两个文档执行安装命令时会遇到相同的失败。本次 PR 仅修补了其中一个文件,导致同类文档之间存在不一致(两个文档仍是错误的旧模式,一个已修正)。

建议:将 examples/qwen3_6/README.md 第 97 行和 examples/mcore/qwen3_5/README.md 第 100 行的 fla-npu_*.run 同样修正为 fla-npu-*.run,保持所有文档中 fla-npu 安装命令一致。

likedislike
ascend-robotascend-robot成员
27 天前 添加了label:ci-pipeline-running
ascend-robotascend-robot成员
27 天前 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
27 天前 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
27 天前 评论:
流水线 PR-pipeline_MindSpeed-MM#4321 [ commitID:82d0ade9 ] 已完成
>>>代码风格自动修复执行成功(无修复内容)
阶段 任务名 状态 详情
编译构建 Build 🛑 >>>
恶意代码检查 Antipoison 🛑 >>>
编码安全与规范检查 Only_doc_commit >>>
codeCheck_pre-commit 🛑 >>>
pre-commit 🛑 >>>
Compatibility_Interface 🛑 >>>
开源片段检查 SCA 🛑 >>>
开发者测试 UT 🛑 >>>
ST 🛑 >>>
UT-pool 🛑 >>>
ST_pool 🛑 >>>
流水线 PR-pipeline_MindSpeed-MM >>>
此流水线已支持下列评论快捷指令,仅PR创建者和白名单成员[wujinyuan1, htwang, hhhzhuyizhi, cxiaolong, bigdog1206, ghoshaw, yaoyaoxu]评论有效
  • compile : 运行流水线
  • retry : 重试流水线所有失败子任务
  • retry <任务名> : 仅重试指定失败子任务
  • stop : 停止流水线
likedislike
htwang成员
27 天前 评论:

/lgtm

likedislike
htwang成员
27 天前 评论:

/approve

likedislike
htwang成员
27 天前 评论:

同意合入

likedislike
ascend-robotascend-robot成员
27 天前 添加了label:approved
ascend-robotascend-robot成员
27 天前 解决了最后一个问题
ascend-robotascend-robot成员
27 天前 合入了pull request
ascend-robot
ascend-robot成员
27 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
ascend-robot
ascend-robot成员
27 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike