Prhub

#36178 [Fix] Keep the MiniCPM-SALA config reads visible to the resolution ratchets

原始 PR 作者 alphabetc1 合并时间 2026-08-25 07:05 文件变更 3 提交数 3 评论 5 代码增减 +24 / -12

执行摘要

修复 MiniCPM-SALA 配置读取对解析棘轮不可见的问题

test/registered/unit/test_chain_read_ratchet.py 在 main 上失败,因为 MiniCPM-SALA 添加的两处读取对解析普查不可见,且第一处掩盖了第二处。具体错误包括:_minicpm_sala_overrides 通过循环变量写入映射,导致 _returned_field_names 无法推导写入字段而抛出 AssertionError: non-literal key in _minicpm_sala_overrides;以及 minicpm/backend.py:276 通过另一对象访问启动记录读取已解析写入的字段,违反棘轮约束。

该 PR 值得精读,因为展示了如何通过调整配置写入与读取方式来满足静态可分析性测试的要求,以及如何配套调整测试发布调度上下文。重点关注 overrides.py 中从循环到字面量键的重构思路,以及 backend.py 中统一读取路径的实践。

讨论亮点

Codex 机器人指出,在 backend.py 中改用 get_schedule() 后,单元测试的构造路径 _construct_sparse_backend 在隔离运行时会因为没有发布调度上下文而抛出 ValueError: config namespace 'schedule' not published,如果其他测试先发布了上下文,则会读取到陈旧值,从而可能使测试失效。这一反馈促使了测试文件的相应调整。

实现拆解

  1. 修改覆盖写入方式:在 python/sglang/srt/arg_groups/overrides.py_minicpm_sala_overrides 函数中,将原来遍历字段名元组并赋值的循环改为逐个使用字面量键 attention_backendprefill_attention_backenddecode_attention_backend 赋值。这样写入的字段集合在静态上可推导,使得棘轮测试能够准确统计。行为上,每个字段仍然仅在配置的后端是 MiniCPM 稀疏后端时才会被覆盖,与原逻辑等价。

  2. 调整配置读取路径:在 python/sglang/srt/layers/attention/minicpm/backend.py 中,将原来从 model_runner.server_args.chunked_prefill_size 读取改为从 get_schedule().chunked_prefill_size 读取,并在 import 中增加 get_schedule。这与 flashattention_backend.py:350triton_backend.py:288 的写法一致,确保读取的是发布后的配置值。

  3. 更新测试配套:在 test/registered/unit/layers/test_minicpm_sparse_metadata.py 中,由于后端现在从调度袋读取 chunked_prefill_size,测试需要在构造后端前发布调度上下文。在 setUp 中通过 get_context().override_server_args(chunked_prefill_size=64) 发布默认值,并在 _construct_sparse_backend 中通过 get_schedule().override(...) 为特定测试覆盖不同的值,同时移除了 model_runner.server_args 中对该字段的假配置。

文件 模块 状态 重要度
python/sglang/srt/arg_groups/overrides.py 配置覆盖 modified 6.12
python/sglang/srt/layers/attention/minicpm/backend.py 注意力后端 modified 5.23
test/registered/unit/layers/test_minicpm_sparse_metadata.py 测试 modified 5.15

关键符号

_minicpm_sala_overrides MiniCPMSparseBackend.__init__ setUp

关键源码片段

python/sglang/srt/arg_groups/overrides.py core-logic

核心修改:将 MiniCPM-SALA 密集后端的覆盖写入从循环变量改为字面量键,使写入字段集合静态可推导,修复棘轮测试的可见性问题。

# python/sglang/srt/arg_groups/overrides.py
@_register_for("MiniCPMForCausalLM", "MiniCPMSALAForCausalLM")
def _minicpm_sala_overrides(server_args: Any, hf_config: Any) -> dict:
    # ... 前略,包含 DP attention 与 hierarchical cache 校验 ...
    if envs.SGLANG_MINICPM_FORCE_DENSE.get():
        dense_backends = {
            "minicpm_flashattn": ("fa4" if is_blackwell_supported() else "fa3"),
            "minicpm_flashinfer": "flashinfer",
        }
        # 使用字面量键写入覆盖,确保写入字段集合可静态推导,
        # 否则循环变量会隐藏写入的字段名,导致棘轮测试无法识别。
        dense_attention = dense_backends.get(server_args.attention_backend)
        if dense_attention is not None:
            overrides["attention_backend"] = dense_attention
        dense_prefill = dense_backends.get(server_args.prefill_attention_backend)
        if dense_prefill is not None:
            overrides["prefill_attention_backend"] = dense_prefill
        dense_decode = dense_backends.get(server_args.decode_attention_backend)
        if dense_decode is not None:
            overrides["decode_attention_backend"] = dense_decode
    # ... 后略,处理稀疏后端路径 ...
    return overrides
python/sglang/srt/layers/attention/minicpm/backend.py dependency-wiring

将 chunked_prefill_size 的读取从 model_runner.server_args 改为 get_schedule() 从调度袋获取,与其他后端保持一致,并触发测试配套调整。

# python/sglang/srt/layers/attention/minicpm/backend.py
from sglang.srt.runtime_context import get_parallel, get_schedule# 在 MiniCPMSparseBackend.__init__ 中:
chunked_prefill_size = get_schedule().chunked_prefill_size
if self.minicpm_fuse_topk and chunked_prefill_size <= 0:
    raise ValueError(
        "MiniCPM fused top-k requires a positive --chunked-prefill-size."
    )
test/registered/unit/layers/test_minicpm_sparse_metadata.py test-coverage

配套测试调整:在 setUp 中发布调度上下文,并在 _construct_sparse_backend 中通过 schedule override 覆盖 chunked_prefill_size,以适配新的读取方式。

# test/registered/unit/layers/test_minicpm_sparse_metadata.py
class TestMiniCPMSparseMetadata(CustomTestCase):
    def setUp(self):
        super().setUp()
        # 后端现在从调度袋读取 chunked_prefill_size,
        # 因此必须在构造任何对象前发布调度上下文。
        override = get_context().override_server_args(chunked_prefill_size=64)
        override.install()
        self.addCleanup(override.restore)# 在 _construct_sparse_backend 中:
with (
    get_schedule().override(chunked_prefill_size=chunked_prefill_size),
    patch.object(backend_module, "MiniCPMHybridConfig", SimpleNamespace),
    # ... 其余 patch ...
):
    backend = MiniCPMSparseBackend(model_runner, use_flashinfer=use_flashinfer)

评论区精华

调度上下文未发布导致测试风险 测试

Codex 机器人指出,在 backend.py 改用 get_schedule() 后,单元测试的构造路径 _construct_sparse_backend 在隔离运行时会因为没有发布调度上下文而抛出 ValueError,如果其他测试先发布了上下文,则会读取到陈旧值,使测试失效。

结论:已在测试文件中添加 setUp 方法发布调度上下文,并通过 get_schedule().override 设置特定值,解决该问题。 · 已解决

风险与影响

风险较低,因为此变更主要影响测试与配置读取路径,没有改变实际行为。但需要注意:如果未来有其他代码路径直接构造 MiniCPMSparseBackend 而未发布调度上下文,会触发 get_schedule() 的异常。此外,测试文件中对 server_args 的修改可能影响其他依赖该假配置的测试用例。

影响范围较小,主要涉及 MiniCPM-SALA 配置相关的测试与内部实现。对用户无影响,对团队而言,修复了 CI 中的棘轮测试,保证了配置解析的可见性约束得到满足。对系统而言,统一了 chunked_prefill_size 的读取方式,提高了代码一致性。

测试上下文依赖 配置读取路径变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论