Skip to content

fix(OmniH2O): 第 9 幕「更新学生」改成从损失连到学生的虚线箭头,只重渲这一幕 - #638

Merged
ImChong merged 2 commits into
mainfrom
claude/omnih2o-details-animation-video
Oct 16, 2026
Merged

ImChong merged 2 commits into
mainfrom
claude/omnih2o-details-animation-video

Conversation

@ImChong

@ImChong ImChong commented Oct 16, 2026

Copy link
Copy Markdown
Owner

接 #636(OmniH2O 笔记、十七幕动画与视频),按看片反馈修第 9 幕「DAgger:学生开车,老师批改」。

问题

  • 学生框左边那条虚线本来要表示「损失回到学生、更新学生的参数」,但代码里四个点只画了前两个:
    • 线停在离学生框 22 个单位的地方;
    • 没有箭头,标签「更新学生」也离得很远。
  • 旁边那条绿色实线(学生自己的动作进损失)没有标签,两条线看起来像一回事。

改动

  • 虚线改成一直画到学生框底边的箭头,小圆点沿着它往上跑,标「梯度回传 / 更新学生」。
  • 绿色实线标上「学生自己的 $\mathbf{a}_t$」。
  • 旁白没变。
  • tests/test_paper_demos.py 守住这支箭头:要用 arrow() 画到学生框,不能再出现 up.slice(0, 2)。

视频

  • 时间线没变,所以只重渲第 9 幕:帧 11,340–12,546,两个 render.mjs chunk 并行,每个约 40 s。
  • 其余部分按帧号拼接上一版的分段,再一次编码成 CRF 19:
    • 接缝两侧新旧帧的平均差 0.12–0.22 灰度级,说明是同一画面;
    • 箭头所在区域只在第 9 幕箭头出场之后有变化,其它时刻和上一版一致。
  • 整片 24,136 帧,音视频时长 804.53 / 804.53 s。网页版 -crf 37 仍是 15.1 MB,已替换笔记 media/ 里的视频,文案里的体积不用改。

工具

  • check_layout.mjs 可以在末尾加时间段,只查改过的几幕,例如 node check_layout.mjs omnih2o story near 378,418。
  • 这次就靠它查出两行新标签之间只隔 1 px,加大行距后才渲的终版;视频 README 已记下这个用法。

检查

  • check_layout.mjs omnih2o story near 378,418:no overlap
  • pytest:305 passed;ruff check scripts/ tests/:通过
  • 发布清单 027 那一行记下了这次发布后的修改。

第 9 幕修复前(左)与修复后(右),取自成片第 398 s:

<img alt="第 9 幕更新学生箭头修复前后" src="https://github.com/ImChong/Robot_Learning_Paper_Notebooks/blob/5a677d9eb330107d467332d24e845b3684565901/.github/pr-assets/pr-omnih2o-scene9-update-arrow.png?raw=true" />

🤖 Generated with Claude Code

https://claude.ai/code/session_019XgFzxdYKHJTJgs5FJfMyB


Generated by Claude Code

- 原来这条虚线只画了半截(四个点只用了前两个)、没箭头,离学生框差 22 个单位,看不出是什么;改成画到学生框的虚线箭头,小圆点往上跑,标「梯度回传 / 更新学生」
- 旁边的绿色实线标上「学生自己的 a_t」,两条线各说各的
- 旁白没变,只重渲帧 11,340–12,546,其余按帧号拼接上一版的分段,接缝新旧帧平均差 0.12–0.22 灰度级;整片 24,136 帧、804.53 / 804.53 s,网页版仍是 15.1 MB
- check_layout.mjs 可在末尾加时间段只查改过的几幕;测试守住这支箭头

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019XgFzxdYKHJTJgs5FJfMyB
@ImChong
ImChong merged commit d0f3c11 into main Oct 16, 2026
1 check passed
@ImChong
ImChong deleted the claude/omnih2o-details-animation-video branch October 16, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants