将 RolloutExecutor.generate 方法重命名为 get,以准确反映其职责。
对于关注系统架构演进或参与相关模块开发的工程师,建议精读此 PR,以理解团队在命名约定和职责划分上的意图。对于日常功能开发,此 PR 影响有限,但可作为未来相关重构的参考。
标签列表
聚合结果
将 RolloutExecutor.generate 方法重命名为 get,以准确反映其职责。
对于关注系统架构演进或参与相关模块开发的工程师,建议精读此 PR,以理解团队在命名约定和职责划分上的意图。对于日常功能开发,此 PR 影响有限,但可作为未来相关重构的参考。
将 RolloutManager 拆分为 InferenceController 与 RolloutExecutor,实现职责分离。
此 PR 是一个有意义的架构重构,值得精读,尤其关注两个新类之间的职责边界划分是否清晰合理。设计决策上,将推理基础设施管理与数据执行流分离,符合单一职责原则,是合理的架构演进。建议结合后续 PR#1843 一起阅读,以获得完整的重构图景。
修复 Dashboard 轨迹面板数据映射错误,确保标签与点击行为指向同一叶子样本。
此 PR 值得精读,特别是对于参与 Dashboard 开发的工程师。它展示了一个典型的“数据一致性”修复模式:当业务数据模型(如支持多叶子的 TITO 索引)发生变化时,需要审视所有消费该数据的组件(如 Map 构建)是否仍然适用。修复本身简洁有效,且附有清晰的解释性注释,是维护代码可读性的好实践。对于理解 PR#2881 的完整影响至关重要。
修复 Dashboard 无法区分共享 sample_index 的 TITO v2 叶子样本,引入 occurrence 键解决数据混淆与 API 500 错误。
该 PR 值得精读。它不是一个简单的 bugfix,而是针对一个设计上的歧义(共享 sample_index)提出的系统性解决方案,贯穿了后端数据模型、API 契约、前端路由和 UI 展示。review 过程中由自动化工具指出并推动修复的两个关键正确性问题(KeyError 和 occurrence 编号错配)极具洞察力,展示了对数据一致性和边缘情况的深入考虑。其引入的 `(sample_index, occurrence)` 键模型和对历史数据的兼容处理方式,对类似场景有很好的借鉴意义。
空 step 补全声明式 schema,修复 dashboard 多页面 500 崩溃
值得精读。改动量不大但信息密度高:polars 空数据 schema 推断陷阱、缓存版本在格式迁移中的作用(mtime 匹配 ≠ 格式有效)、并发读下的原子写、以及「先用 revert 实验证明测试能抓到 bug 再提交」的验证方法(PR body 中 2 failed 的对照表)。对 dashboard 维护者、做数据管道容错的工程师尤其有参考价值;`blank_samples` 夹具和 `test_pre_fix_columnless_cache_is_not_served` 是两段值得模仿的测试写法。
Rollouts 标签落在首个可读 step,修复实时点击报错
值得精读。改动虽小但完整呈现了一个前端修复的工程方法论:先枚举全部失败形态再决定统一处理策略(threw 与 empty 同等对待)、为回退探测设置语义边界(5 步后停止并暴露真实错误)、处理异步间隙的竞态守卫、考虑历史栈与重试预算。“有界回看 + 显式请求不重定向”的组合对同类 dashboard 导航很有借鉴意义。若后续要为 dashboard 前端补测试设施,本 PR 的 Playwright 验证矩阵(3 种 dump × 4 种检查)是现成的回归用例清单。
令牌条改为整条滚动,分页控件移除,请求减半
值得精读,尤其是关心 dashboard 前端与超长列表渲染的工程师。作者先用实测证明 DOM 构建成本远低于直觉(64k span 仅 66 ms),再用 `offsetParent + scrollTop` 的懒定位替代分页状态,最后针对 review 发现的隐藏面板时序问题设计一次性 `_onShown` 回调;这三个决策都可迁移到其他长列表或懒加载面板场景。
原始 PR · 作者 yushengsu-thu · 合并时间 2026-08-22 09:09
Dashboard 新增 AMD SMI 遥测,NVML 优先自动回退
值得精读,尤其是 Provider 抽象、单位差异处理与降级语义的设计:`_AmdSmiProvider` 与 `_NvmlProvider` 共享同一接口而实现细节差异被干净隔离,`_amd_socket_power` 对多代 ROCm 字段兼容的思路可复用于其他硬件平台。创建类似硬件适配层时可参考本 PR 的 importlib 延迟导入与 `sys.modules` 打桩测试方式。