Prhub

#1773 refactor: remove slime router

原始 PR 作者 zhuzilin 合并时间 2026-03-27 16:57 文件变更 22 提交数 5 评论 0 代码增减 +36 / -604

执行摘要

移除过时的 slime router,统一使用 sglang_router 以简化架构。

从slime/utils/arguments.py的patch中可见,添加了警告:"--use-slime-router is deprecated and ignored. slime now always uses sglang_router built from https://github.com/zhuzilin/sgl-router." 这表明slime router已过时,被sglang_router取代,旨在简化系统架构并统一依赖。

建议阅读slime/ray/rollout.py中的_start_router函数变更,以理解router启动逻辑的统一化设计;同时关注slime/utils/arguments.py的弃用处理,学习向后兼容的最佳实践。此PR值得精读,因为它揭示了架构演进方向。

讨论亮点

本次PR没有review评论或讨论,基于提供的材料,comments_count和review_comments_count均为0,因此无额外技术交锋或争议点。

实现拆解

实现方案分为三部分:

1) 代码移除:删除slime/router/router.py等核心实现文件;
2) 逻辑更新:修改slime/ray/rollout.py中的_start_router函数,移除slime router路径,只启动sglang_router;
3) 配置清理:在slime/utils/arguments.py中移除slime router相关参数并添加弃用警告,同时更新文档和脚本以移除--use-slime-router标志。

文件 模块 状态 重要度
slime/router/router.py router removed 8.0
slime/ray/rollout.py rollout modified 7.0
slime/utils/arguments.py utils modified 6.0

关键符号

_start_router

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

评论区精华

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

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

风险与影响

技术风险包括:

1) 向后兼容性:移除了--use-slime-router参数,但代码中已设置警告并自动禁用该标志,降低风险;
2) 功能缺失:slime router的特定功能(如PD disaggregation)需确保sglang_router完全支持,从patch看sglang_router已集成相关逻辑;
3) 文档更新滞后:删除了slime-router.md文档,但其他文档更新可能不完整,需检查相关引用。风险点具体在slime/utils/arguments.py的参数移除和slime/ray/rollout.py的路由启动逻辑变更。

影响范围:

1) 用户影响:用户需要更新脚本移除--use-slime-router标志,否则会收到弃用警告,但功能不受影响;
2) 系统影响:简化了路由架构,减少代码维护成本,提升一致性;
3) 团队影响:工程师需熟悉sglang_router配置,但降低了slime router的维护负担。影响程度中等,主要涉及配置和文档调整。

向后兼容性风险 文档更新滞后 依赖变更风险

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论