重置 vLLM 多模态和编码器缓存
值得阅读。本 PR 修复了多模态 rollout 中一个实用的缓存一致性 Bug,设计上通过扩展现有方法而非引入新入口点,保持了架构整洁。虽然版本检查方式有争议,但当前实现对于已知 vLLM 版本是安全的。
verl: Volcano Engine Reinforcement Learning for LLMs
重置 vLLM 多模态和编码器缓存
值得阅读。本 PR 修复了多模态 rollout 中一个实用的缓存一致性 Bug,设计上通过扩展现有方法而非引入新入口点,保持了架构整洁。虽然版本检查方式有争议,但当前实现对于已知 vLLM 版本是安全的。
原始 PR · 作者 fuergaosi233 · 合并时间 2026-06-02 11:06
修复 NaiveRouter 请求失败后 worker 计数泄漏
建议审批前要求作者修复 body 读取的泄漏问题,并补充针对此修复的单元测试。代码整体方向正确,设计思路值得学习。
替换 Qwen2.5 CI 测试为 Qwen3 并添加基线验证
如果关注 NPU CI 的演进和模型升级策略,可以快速浏览该 PR。但在 shell 脚本质量方面参考价值有限(未采纳自动化建议)。建议后续修复硬编码路径问题。
修复 Megatron ref 模型 GPU 显存泄漏
建议精读该 PR,特别是 `_can_safely_resize_storage` 函数的逻辑和同步拷贝 + `resize_(0)` 的使用模式。该修复对于避免大模型训练中的显存泄漏至关重要。但建议补充单元测试覆盖非 DDP 路径的卸载场景。
简化 VeOmni 安装测试,移除对 sys.path 依赖
值得快速合并,修复 CI 失败,提高测试可靠性。不要求深入 review。
版本号从 0.8.0.dev 升至 0.9.0.dev
此 PR 为简单的版本 bump,无需深度阅读。可作为版本管理流程的参考。
FSDP actor/critic CI 配置校验修复
该 PR 改动极小且直接,可快速合入以修复 CI。但 review 中提出的 `model_config` 类型兼容性问题值得在后续迭代中修复,以增强鲁棒性。
原始 PR · 作者 wenzhaoabc · 合并时间 2026-06-01 10:30
支持 trl >=0.29 时导入 AutoModelForCausalLMWithValueHead
简单但必要的兼容性修复,值得快速合并。可作为依赖升级兼容性处理的范例。
参与讨论