sft 教程 issue修复-cann-learning-hub-AtomGit

已合并

sft 教程 issue修复 #521

hanboyou创建于 3 天前从mystri/cann-learning-hub： sft-intermediate 合入到 cann/cann-learning-hub： test

sft 教程 issue修复 #521

已合并

hanboyou创建于 3 天前

从mystri/cann-learning-hub sft-intermediate合入到cann/cann-learning-hub test

hanboyou成员

3 天前

Issue	所属范围	完成情况	核验说明
#222	SFT	已完成	已重写 `01.03_sft_concepts.ipynb` 的评估结构，突出格式分是 SFT 首要目标，并修复标点、重复和指标表述。
#226	SFT	已完成	已重制第三章训练流程图与 vf-eval 流程图；结束条件和反馈回环已修正；参数说明改为 8 局、最多 48 次生成。
#245	SFT	已完成	clone 单元已支持重复执行，会校验 origin、识别非 Git 半成品目录并输出 commit；课程固定到已验证的 `torchtitan-npu` 提交 `f8b72f1`，重复执行及未来 `master` 更新都不会改变课程版本。已有本地源码修改时不会强制 checkout。
#377	SFT	已完成	`02.03_environment_setup.ipynb` 已补齐第 4 节和第 6 节标题，目录编号现为连续的 1～7。
#291	SFT	已完成（SFT 范围）	目标 Notebook 的答案输出已清理；全部 42 个 SFT Notebook 均为有效 JSON，且不存在带输出或执行计数的代码单元。其他课程不纳入本分支要求。
#445	SFT	已完成	已按固定 `verifiers` 提交的真实 CLI 修正：保留有效的 `--max-tokens 1024`，不传该版本不存在的 `--provider`；增加未注册的本地模型名，防止 endpoint 注册表覆盖 `--api-base-url`。正文已解释 OpenAI 兼容客户端的选择方式。
#446	SFT/RL 兼容性	已完成（SFT 范围）	SFT 环境已改为 `pyarrow==24.0.0` 并增加版本断言；只修改 SFT 即完成与 RL 既定版本的对齐。
#452	SFT，附带 RL 建议	已完成（SFT 范围）	05.03 图片已引用；第 5 章图片统一为 SVG 优先、PNG 回退；SFT 内其他零引用图片均已引用。RL 图片去重建议超出范围，无需处理。
#409	SFT	已完成	`01.03_sft_concepts.ipynb` 中“阶段阶段”、不完整句子和双句号均已修正。
#495	SFT	已完成	`training_curves.png`、两张融合算子链图及 SFT `cann-lab-env.png` 均已在对应正文中引用。
#514	SFT	已完成	05.04 表格中的特殊 token 已写为 `<\|im_end\|>`，不会再被 Markdown 解析成额外列。

Pull Request已成功合入, 合并人@CANN-robot

（感谢 hanboyou 的贡献）

hanboyou成员

3 天前创建了 pull request，commit 6e5411e4

atomgit-bot

3 天前评论：

变更摘要

此 PR 主要对 SFT 训练流水线教程进行多方面的完善：修正 notebook 中已过时的执行输出与元数据，统一升级图片标签为支持 SVG/PNG 回退的 <picture> 元素，改进 torchtitan 环境配置脚本的健壮性并升级 pyarrow 依赖版本，同时更新评测流程与参数说明，明确 SFT 与 RL 阶段的职责划分。

主要改动

02.03_environment_setup.ipynb 环境配置脚本加固：将 git clone 改为带远程仓库校验、增量 fetch/merge、本地修改保护的条件更新逻辑，避免重复克隆或误用错误仓库；同时将 pyarrow 从 21.0.0 升级到 24.0.0 并添加版本断言。
03.04_inference_eval.ipynb 评测命令与参数说明修正：vf-eval 命令补全 --provider openai 和 --max-tokens 1024 参数，并将 rollout 统计从错误的"共 60 条"修正为"共 8 局，最多 48 次生成请求"。
图片标签统一为 <picture> 元素：在 03.01_chapter_intro.ipynb、03.04_inference_eval.ipynb、04.02_adding_fused_operator.ipynb、04.04_profiling_output_analysis.ipynb、05.03_attention_kernel_benchmarking.ipynb 等多处将纯 <img> 改为 <picture> 标签，提供 SVG 优先、PNG 回退的响应式图片支持。
Notebook 执行元数据清理：批量清除 03.04、04.02、04.03、04.04、04.05 等多个 notebook 单元格的 execution_count 与过时 outputs，确保文档呈现为干净的可执行状态。
SFT 评估概念与图片路径修正：在 01.03_sft_concepts.ipynb 中重写 SFT 评估职责描述，明确格式分为首要目标、猜中率为基线记录；修复 05.02_attention_kernels.ipynb 和 05.05_torchtitan_attention_ops.ipynb 中 <img> 的 src 从 .svg 错误指向为 .png。

不准确？

atomgit-bot

3 天前评论：

代码审查

✅ 未发现问题

不准确？

CANN-robot成员

3 天前添加了label：stat/needs-squash

CANN-robot成员

3 天前添加了label：cann-cla/yes

CANN-robot成员

3 天前评论：

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-cann/cann-learning-hub	✅ 张德鹏, waimaidaole (2/2)	✅ 张德鹏, waimaidaole (2/1)

💡 Tip:

Committer can comment /approve or /lgtm

Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

hanboyou成员

3 天前修改标题为 “sft 教程 issue修复”，原标题为“Merge remote-tracking branch 'mystri/sft-intermediate' into sft-intermediate”

此处折叠了11条事件消息查看更多

hanboyou成员

3 天前修改了pull request 的描述

meme成员

3 天前评论：

/lgtm
/approve

CANN-robot成员

3 天前添加了label：approved

张

张德鹏成员

3 天前评论：

/lgtm
/approve

CANN-robot成员

3 天前添加了label：lgtm

CANN-robot成员

3 天前合入了pull request