Prhub

sgl-project/sglang · 标签视图

标签列表

聚合结果

debugging 相关 PR

2026-08-30
缺陷修复 重要性 7.25 洞察度 6.00

空闲循环树缓存深度校验默认关闭,修复 DP 组调度卡死

值得精读。代码量虽小,却包含两个可复用设计:一是用 EnvField 的 callable default 实现「生产默认关、CI 默认开、显式覆盖优先」的三态配置语义,让调试仪器默认不进生产路径同时不牺牲 CI 回归;二是测试通过 contextmanager 清理进程环境变量来稳定默认值断言,是环境耦合测试的典型解法。对调度器与缓存团队,建议同时关注 sanity_check 自身性能(TODO(hzh))与文档同步。

2026-08-21

#34406 TP/PP Consensus checker

原始 PR · 作者 stepinto · 合并时间 2026-08-21 01:36

功能 重要性 9.00 洞察度 6.00

新增 PP/TP 秩间共识检查器,提前定位分歧根因

值得精读,重点看 `rank_consensus_checker.py` 的装饰器设计:选择器归一化、`self` / `cls` 跳过、关闭时 import 阶段短路实现零开销、后台线程 + 专用 gloo 组的比对模型。建议与 #27010(HiCache PP 一致性修复)一起阅读,理解 PP 分歧的根因与验证链路。后续可关注 hzh0425 提出的 e2e 测试 follow-up。

2026-08-20
功能 重要性 7.95 洞察度 5.00

加载日志按内存类型报告组件权重驻留位置

值得精读。虽然只是日志增强,但 `component_residency_bytes()` 的实现有几个可复用的设计:用 `/proc/self/maps` 区分 file-backed 与 anonymous host 内存、用 storage 指针去重避免扁平 buffer 重复计数、pinned 判断先于 file-backed 判断(因为 CUDA host allocator 背后是命名映射)。对任何需要诊断显存/内存占用的模块都有参考价值。

2026-08-17
缺陷修复 重要性 6.81 洞察度 4.00

抑制 FlashInfer TRT-LLM workspace 预期警告

该 PR 值得精读,它是一个典型的“抑制外部库噪音警告但保持语义不变”的实践。设计上通过缓存元数据并在决策路径上提前短路,既避免了无效调用又保留了所有合法复用场景,测试覆盖也很全面(区分了快速路径与验证器调用场景)。对于需要与第三方库日志噪音做斗争的项目有很好的参考价值。

2026-08-14
重构 重要性 6.96 洞察度 4.00

重构 environ.py,560 个环境变量归入 64 个主题区块

值得快速浏览 `environ.py` 类头部的组织原则注释,理解这个纯组织重构的规范化方法;作者用整文件 AST 对比证明零行为变化,是这类大规模移动重构的示范做法。由于是纯顺序调整,不需要深入 review 每个字段,但建议后续新增环境变量时遵守区块纪律,避免重构效果衰减。

2026-08-12
缺陷修复 重要性 5.71 洞察度 4.00

统一 tokenizer 警告过滤器安装,补齐 processor 路径

值得快速阅读,作为小范围 bugfix 的样板:它展示了如何用模块级单例加统一后处理入口来收敛跨路径的行为修补。没有需要深入研究的算法或架构,后续可在 tokenizer 加载链路补一个针对警告过滤行为的单元测试,防止回归。

功能 重要性 6.93 洞察度 4.00

增强位精确融合回退诊断并新增 FlashInfer 提示

值得精读,尤其是维护 diffusion 融合 kernel 或位精确验证机制的工程师。重点关注三点设计:一是用“回调传入 + 函数内延迟导入”实现零稳态开销的诊断;二是用多层 try/except 保证诊断永远不破坏正确性回退;三是用 `importlib.metadata` 对多个关联包版本做 best-effort 探测。对依赖日志解析与告警的团队,建议同步检查 warning 文案变化是否需要适配。

2026-08-10

#34169 Add skill for the logprob consistency tests

原始 PR · 作者 ispobock · 合并时间 2026-08-10 01:33

文档 重要性 4.75 洞察度 5.00

新增 KL 一致性测试技能文档,沉淀调试与阈值方法论

建议负责确定性推理、logprob 一致性测试与相关 CI 的工程师精读,重点是两个独立条件的区分与 MoE 放大效应部分;对一般读者可略过。该文档是团队调试经验的优质沉淀,值得保留,并建议在 `kl_test_utils` 或确定性推理代码变动时同步更新。