Prhub

#24384 misc: add marlin to moe runner choices; drop dead env var doc

原始 PR 作者 hnyls2002 合并时间 2026-05-05 06:01 文件变更 2 提交数 2 评论 2 代码增减 +1 / -5

执行摘要

修复 marlin 缺失及文档清理

PR body 指出 MOE_RUNNER_BACKEND_CHOICES 缺少 marlin,但 MoeRunnerBackend.MARLIN 枚举自 #14554 就已存在,且文档已指示用户使用 --moe-runner-backend marlin,但 argparse 会拒绝该值;同时 SGLANG_DISABLE_REQUEST_LOGGING 环境变量在代码库中已无读取者,因此删除相关文档行。

建议合并。该 PR 属于典型的微小修复,改动量小且明确,无需深度 review。

讨论亮点

Review 中无实质性讨论,仅含一条自动评论,无争议。

实现拆解

  1. python/sglang/srt/server_args.py: 在 MOE_RUNNER_BACKEND_CHOICES 列表末尾添加字符串 "marlin",使 argparse 能识别该选项,与枚举定义保持一致。
  2. docs_new/docs/references/environment_variables.mdx: 删除已废弃环境变量 SGLANG_DISABLE_REQUEST_LOGGING 对应的表格行(共 5 行),避免误导用户。
文件 模块 状态 重要度
python/sglang/srt/server_args.py 服务参数 modified 4.3
docs_new/docs/references/environment_variables.mdx 文档 modified 2.64

关键源码片段

python/sglang/srt/server_args.py core-logic

修复了 CLI 参数校验缺失 marlin 选项的 bug,使 --moe-runner-backend marlin 可用

# MOE_RUNNER_BACKEND_CHOICES = [...] # 原有后端列表
MOE_RUNNER_BACKEND_CHOICES = [
    "auto",
    "deep_gemm",
    "triton",
    "triton_kernel",
    "flashinfer_trtllm",
    "flashinfer_trtllm_routed",
    "flashinfer_cutlass",
    "flashinfer_mxfp4",
    "flashinfer_cutedsl",
    "cutlass",
    "aiter",
    "marlin", # 新增:补齐枚举定义,使 CLI 参数 --moe-runner-backend marlin 可用
]

评论区精华

没有提炼出高价值讨论线程

当前评论区没有形成足够清晰的争议点或结论,后续有更多讨论时会体现在这里。

风险与影响

风险极低。marlin 的添加仅扩展了合法参数列表,不会影响现有行为;文档删除仅移除了无读取者的废弃变量。

直接影响:用户现在可以通过 --moe-runner-backend marlin 启用 Marlin MoE 后端,而不会被 argparse 拒绝;文档更加准确。间接影响:无。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论