Prhub

#21250 [diffusion] Fix torch.zeros typo in causal wan

原始 PR 作者 BBuf 合并时间 2026-03-24 14:39 文件变更 1 提交数 1 评论 3 代码增减 +1 / -1

执行摘要

修复扩散模型中 torch.zeroes 到 torch.zeros 的 typo,避免潜在错误。

修复 torch.zeroes 到 torch.zeros 的 typo,以确保 PyTorch 函数的正确使用,避免可能引发的运行时错误或功能异常。从 commit 消息 'diffusion: fix torch.zeros typo in causal wan' 推断,动机是纠正一个明显的代码错误。

该 PR 变更简单,不值得精读,但可作为代码审查中注意 typo 的示例。对于新贡献者,可参考此类修复了解代码规范。

讨论亮点

PR 中没有进行代码审查讨论,提交后直接触发 CI 测试(通过 /tag-and-rerun-ci 命令),并自动合并。这表明变更被视为琐碎修复,无争议或设计权衡。

实现拆解

仅修改一个文件的一行代码:在 python/sglang/multimodal_gen/runtime/models/dits/causal_wanvideo.py 的 forward 函数中,将 null_shift = null_scale = torch.zeroes(...) 更正为 torch.zeros(...)。未涉及其他逻辑或结构变更。

文件 模块 状态 重要度
python/sglang/multimodal_gen/runtime/models/dits/causal_wanvideo.py multimodal_gen/diffusion modified 1.0

关键符号

forward

分析完成后,这里会展示 LLM 生成的相对完整源码片段和详细注释。

评论区精华

无 review 讨论 other

PR 提交后,作者 BBuf 使用 /tag-and-rerun-ci 触发 CI 测试,没有代码审查评论或讨论。

结论:变更被视为琐碎修复,直接合并。 · 已解决

风险与影响

风险极低:仅为 typo 修复,不改变逻辑或行为;torch.zeros 是 PyTorch 标准函数,替换正确。潜在风险是如果 torch.zeroes 有特殊用途(但无证据),可能引入兼容性问题,但基于上下文,这是明显错误。

对用户无直接影响;对系统影响:修复了扩散模型中的潜在 bug,提升代码正确性;对团队影响:无,变更简单且通过 CI 验证。

typo 修复

关联 Issue

未识别关联 Issue

当前没有检测到明确关联的 Issue 链接,后续同步到相关引用后会出现在这里。

完整报告

参与讨论