已关闭
fix(modeling/phi3): repair bench/prof shell scripts #589
不喜欢雨天ༀ创建于 12 天前关闭于 5 天前
fix(modeling/phi3): repair bench/prof shell scripts #589
已关闭
不喜欢雨天ༀ创建于 12 天前关闭于 5 天前
不喜欢雨天ༀ
12 天前

变更说明

修复 #123 所报 benchmark/profile 脚本不可执行问题:

  1. HTML 实体污染:bench / prof 两个脚本中大量 $ 被写成 HTML 实体形式,所有变量展开失效——统一还原为字面 $
  2. 输入文件路径错误:两脚本引用不存在的脚本目录下 sample_inputs.txt,修正为实际存在的 ../sample_inputs/input_prompt_small.txt(与输出长度匹配的小样本);
  3. 内嵌 python 引号冲突:bench 的对比汇总段原为双引号 python -c 内嵌含双引号 f-string,bash 解析断裂——改为单引号定界 heredoc,报告路径经环境变量传入。

已通过 bash -n 语法校验。

Fixes #123

likedislike
当前Pull Request已关闭, 关闭人@不喜欢雨天ༀ
不喜欢雨天ༀ不喜欢雨天ༀ
12 天前 创建了 pull request,commit 75442dd3
不喜欢雨天ༀ不喜欢雨天ༀ
12 天前 关联了issue:[Bug-Report]: 3 个 benchmark/profile 脚本无法执行(HTML 实体污染 $、内嵌 python 双引号冲突)
CANN-robotCANN-robot成员
12 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
12 天前 评论:

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/pypto-gym ❌ (0/2)(You can also ask: 尹杰, 孙文博, xinweiliu, li-shengxian3, wangzheng97) ❌ (0/1)(You can also ask: codehjp423, jason_yuan_ye, gaoxiang618, gaoxingwang, xinweiliu)

💡 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
不喜欢雨天ༀ
不喜欢雨天ༀ
10 天前 评论:

/compile

likedislike
CANN-robotCANN-robot成员
10 天前 添加了label:ci-pipeline-running
CANN-robot
CANN-robot成员
10 天前 评论:

流水线任务触发成功
任务链接 [c897f63aae8341dfb5e1015bd479f91d][流水线指导]

任务名称状态日志下载链接
SCA ✅ SUCCESS >>>>>
antipoison ✅ SUCCESS >>>>>
Check_Pr ✅ SUCCESS >>>>>
pre_comment ✅ SUCCESS >>>>>
precommit ✅ SUCCESS >>>>>
codecheck_style ✅ SUCCESS >>>>>
codecheck ✅ SUCCESS >>>>>
Smoke_A900 ❌ FAILED >>>>> >>>>>
likedislike
CANN-robotCANN-robot成员
10 天前 删除了label:ci-pipeline-running
CANN-robotCANN-robot成员
10 天前 添加了label:ci-pipeline-failed
不喜欢雨天ༀ
不喜欢雨天ༀ
7 天前 评论:

以下 5 个文档/脚本修复 PR 已 CI 通过:

  • !589 fix(modeling/phi3): repair bench/prof shell scripts
  • !593 fix(gutenocr_3b): repair bench script quoting
  • !576 fix(scripts): repair three broken bench/prof scripts
  • !597 docs(test): 修正 tests/README 失效链接
  • !590 docs(tests): fix stale qwen3_next test link
  • !598 docs: fix broken relative-link anchors

见您近期在积极评审合入,恳请检视,谢谢!@congqiannan @wangzheng97

likedislike
不喜欢雨天ༀ
不喜欢雨天ༀ
5 天前 评论:

本 PR 的修复内容已完全包含于 !610(文件交集 2 个全覆盖),收敛到 !610 统一处理,关闭本 PR 减少评审队列负担。

likedislike
不喜欢雨天ༀ不喜欢雨天ༀ
5 天前 关闭了 pull request