Prhub

#24093 [CI] Fix black formatting on main (unblocks PR #21247 lint)

原始 PR 作者 Kangyan-Zhou 合并时间 2026-04-30 05:59 文件变更 3 提交数 1 评论 1 代码增减 +7 / -9

执行摘要

运行 black 格式化修复 lint

main 分支上三个文件的格式与 black-jupyter pre-commit 钩子不匹配,导致 lint CI 持续失败,影响所有基于 main 的 PR(如 #21247)。运行 black 后 diff 与 CI 报告完全一致,修复此阻塞问题。

直接合并,无需精读。可关注 black 版本的 CI 配置一致性,防止类似问题。

讨论亮点

无 review 讨论。只有一个机器人评论提示 quota 限制,无实质内容。

实现拆解

  1. python/sglang/srt/model_executor/model_runner.py 中,将 if sv.data_ptr() not in {...}: 集合表达式合并为单行,删除多余换行。
  2. python/sglang/srt/model_loader/loader.py 中,两处函数调用参数格式调整:_transfer_via_nixl 调用合并到一行;分割的日志字符串 "ModelExpress [nixl]: transferred %d tensors, " "%.2f GB in %.2fs" 拼接为一行。
  3. python/sglang/srt/server_args.py 中,if self.modelexpress_source and self.modelexpress_transport == "transfer_engine": 条件语句根据 black 规则添加括号换行,使逻辑结构更清晰。
  4. 运行 pre-commit run black-jupyter --files <files> 验证通过,无行为变更。
文件 模块 状态 重要度
python/sglang/srt/model_executor/model_runner.py 模型执行器 modified 4.21
python/sglang/srt/model_loader/loader.py 模型加载器 modified 4.21
python/sglang/srt/server_args.py 服务参数 modified 4.04

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

评论区精华

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

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

风险与影响

无行为变更,仅格式化调整。风险极低,但需确认 black 版本与 CI 环境一致(26.1.0)以避免后续再次偏差。

影响范围:仅三个源码文件的空白和换行格式。对用户无影响;对开发流程解除 lint 阻塞,使 PR #21247 等分支可正常通过 lint 检查。团队:降低 CI 噪音,提高开发效率。

无行为变更

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论