执行摘要
- 一句话:修复 marlin 缺失及文档清理
- 推荐动作:建议合并。该 PR 属于典型的微小修复,改动量小且明确,无需深度 review。
功能与动机
PR body 指出 MOE_RUNNER_BACKEND_CHOICES 缺少 marlin,但 MoeRunnerBackend.MARLIN 枚举自 #14554 就已存在,且文档已指示用户使用 --moe-runner-backend marlin,但 argparse 会拒绝该值;同时 SGLANG_DISABLE_REQUEST_LOGGING 环境变量在代码库中已无读取者,因此删除相关文档行。
实现拆解
python/sglang/srt/server_args.py: 在 MOE_RUNNER_BACKEND_CHOICES 列表末尾添加字符串 "marlin",使 argparse 能识别该选项,与枚举定义保持一致。
docs_new/docs/references/environment_variables.mdx: 删除已废弃环境变量 SGLANG_DISABLE_REQUEST_LOGGING 对应的表格行(共 5 行),避免误导用户。
关键文件:
python/sglang/srt/server_args.py(模块 服务参数;类别 source;类型 core-logic): 修复了 CLI 参数校验缺失 marlin 选项的 bug,使 --moe-runner-backend marlin 可用
docs_new/docs/references/environment_variables.mdx(模块 文档;类别 other;类型 core-logic): 删除已无代码读取者的废弃环境变量文档,避免误导
关键符号:未识别
关键源码片段
python/sglang/srt/server_args.py
修复了 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 可用
]
评论区精华
Review 中无实质性讨论,仅含一条自动评论,无争议。
风险与影响
- 风险:风险极低。
marlin 的添加仅扩展了合法参数列表,不会影响现有行为;文档删除仅移除了无读取者的废弃变量。
- 影响:直接影响:用户现在可以通过
--moe-runner-backend marlin 启用 Marlin MoE 后端,而不会被 argparse 拒绝;文档更加准确。间接影响:无。
- 风险标记:暂无
关联脉络
- PR #14554 (推测)引入 MoeRunnerBackend.MARLIN 枚举: marlin 枚举由此 PR 引入,但 CLI choices 未同步更新,本 PR 补全了这一遗漏。
参与讨论