执行摘要

• 一句话：修复 debug-train-only 模式下 spec metrics 读取失败 bug
• 推荐动作：建议立即合并。这是一个低风险、高价值的 bugfix，修复了文档中明确推荐的调试模式中的崩溃问题。虽然改动微小，但解决了实际痛点，值得注意其防御性编程模式（使用 getattr 处理可选命令行参数）。

功能与动机

--debug-train-only 模式会跳过 sglang 参数解析（slime/utils/arguments.py 第1414行），导致 args 上未设置 sglang_speculative_algorithm 属性。但 _compute_spec_metrics（slime/ray/rollout.py 第1238行）直接访问该属性，在第一次 rollout 完成时引发 AttributeError。该 bug 由 #640 引入，影响所有使用该调试模式的训练流程。PR body 明确描述了复现场景和修复动机。

实现拆解

该 PR 仅涉及一个文件的单行改动，实现极其简洁：

1. 定位问题: 在 slime/ray/rollout.py 的 _compute_spec_metrics 函数中，第 1239 行原代码 if args.sglang_speculative_algorithm is None 假设该属性一定会存在，但在 --debug-train-only 模式下该属性未被设置。

2. 修复方法: 将直接属性访问改为安全的 getattr 调用：if getattr(args, "sglang_speculative_algorithm", None) is None。这样当属性缺失时，getattr 返回默认值 None，条件为真，函数立即返回空字典 {}，逻辑与非 spec 场景的行为一致。

3. 无测试配套改动: PR 作者注明这是单行防御性修改，逻辑无变化，且已在生产环境验证修复有效（在 2x p5e.48xlarge 上运行 Qwen3.5-35B-A3B SFT 超过 42 step 无崩溃）。

关键文件：

• slime/ray/rollout.py（模块 Rollout 模块；类别 source；类型 core-logic；符号 _compute_spec_metrics）: 修复核心函数 _compute_spec_metrics 中直接访问 args.sglang_speculative_algorithm 导致的 AttributeError，使用 getattr 提供安全默认值。

关键符号：_compute_spec_metrics

关键源码片段

slime/ray/rollout.py

修复核心函数 _compute_spec_metrics 中直接访问 args.sglang_speculative_algorithm 导致的 AttributeError，使用 getattr 提供安全默认值。

def _compute_spec_metrics(args, all_samples: list[Sample]):
    # 使用 getattr 安全读取可选命令行参数：
    # 在 --debug-train-only 模式下，sglang 参数解析被跳过，
    # args 上不会设置 sglang_speculative_algorithm 属性。
    # 若直接访问 args.sglang_speculative_algorithm 会触发 AttributeError。
    # getattr 在属性缺失时返回 None，与原始非 spec 场景行为一致。
    if getattr(args, "sglang_speculative_algorithm", None) is None:
        return {} # 非 spec 模式或属性缺失时，返回空指标
​
    num_samples = len(all_samples)
    metrics = {}
    metrics["spec_accept_rate"] = sum(
        sample.spec_info.spec_accept_rate for sample in all_samples
    ) / num_samples
    metrics["spec_accept_length"] = sum(
        sample.spec_info.spec_accept_length for sample in all_samples
    ) / num_samples
    return metrics

评论区精华

该 PR 没有 review 评论。但由于 PR body 和 commit message 非常详细地描述了问题背景、根因、修复方案和测试验证，可视为自文档化的变更。

• 暂无高价值评论线程

风险与影响

• 风险：风险极低：
  • 改动仅为一行，将直接属性访问替换为 getattr，语义等价（当属性存在时行为不变）。
  • 仅影响 --debug-train-only 模式下 spec metrics 的计算路径，不影响正常训练或 rollout。
  • 无回归风险，因为原逻辑仅是空值检查，getattr 是 Python 标准库的安全模式。
• 影响：影响范围有限但实用：
  • 用户: 修复了 --debug-train-only 模式下的崩溃问题，使该调试模式恢复正常使用。
  • 系统: 无性能或功能影响。
  • 团队: 减少了开发者调试时的阻碍，提升了 --debug-train-only 的可靠性。
  • 风险标记：缺少测试覆盖

关联脉络

• PR #640 MTP training support: 引入了 _compute_spec_metrics 函数并直接读取 args.sglang_speculative_algorithm，是当前 bug 的引入 PR。
• PR #1890 Add missing metrics to log: 最近修改了 slime/ray/rollout.py 中的指标逻辑，与当前文件范围相关。