Prhub

#24095 [misc] fix lint in main branch

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

执行摘要

修复 main 分支 lint 问题

PR 标题和 body 仅提及 "fix lint in main branch",无关联 issue 或更详细的动机描述。推测是 CI 或预提交钩子检测到代码格式问题,需要修复以保持代码风格一致。

无特殊阅读价值,但可用于了解项目的代码风格期望。

讨论亮点

无 review 评论或讨论。

实现拆解

  1. python/sglang/srt/model_loader/loader.py 中,将 _transfer_via_nixl 调用从一个多行调用合并为单行;同时修复了一个 logger 字符串拼接中的多余换行。
  2. python/sglang/srt/model_executor/model_runner.py 中,将 set 推导式的多行写法压缩为单行。
  3. python/sglang/srt/server_args.py 中,将一个多行 if 条件中的括号对齐方式从跨两行调整为一行内换行。
文件 模块 状态 重要度
python/sglang/srt/model_loader/loader.py 模型加载 modified 4.21
python/sglang/srt/model_executor/model_runner.py 模型执行 modified 4.21
python/sglang/srt/server_args.py 服务器参数 modified 3.76

关键符号

load_model_from_modelexpress _transfer_via_nixl _build_nixl_worker_metadata remote_instance_weight_loader_use_transfer_engine

关键源码片段

python/sglang/srt/model_loader/loader.py other

主要 lint 修复所在:合并了函数调用括号的换行,以及修正 logger 字符串拼接的换行。

# python/sglang/srt/model_loader/loader.py
# 变更前:
# self._transfer_via_nixl(
# model, nixl_mgr, source_worker, tp_rank
# )
# 变更后:合并为单行 ( 符合 PEP 8 长度限制 )
self._transfer_via_nixl(model, nixl_mgr, source_worker, tp_rank)# 另一个变更:修复 logger 字符串拼接换行
# 变更前:
# logger.info(
# "ModelExpress [nixl]: transferred %d tensors, "
# "%.2f GB in %.2fs",
# 变更后:去掉多余的换行
logger.info(
    "ModelExpress [nixl]: transferred %d tensors, " "%.2f GB in %.2fs",
    matched,
    total_bytes / 1e9,
    duration,
)
python/sglang/srt/model_executor/model_runner.py other

修复了 set 推导式的多行写法为单行。

# python/sglang/srt/model_executor/model_runner.py
# 在 _build_nixl_worker_metadata 方法中
# 变更前:
# if sv.data_ptr() not in {
# v.data_ptr() for v in model_tensors.values()
# }:
# 变更后:合并为单行
if sv.data_ptr() not in {v.data_ptr() for v in model_tensors.values()}:
    model_tensors[f"{name}.__storage"] = sv
python/sglang/srt/server_args.py other

修复了多行 if 条件的括号对齐方式。

# python/sglang/srt/server_args.py
# 在 remote_instance_weight_loader_use_transfer_engine 方法中
# 变更前:
# if self.modelexpress_source and self.modelexpress_transport == "transfer_engine":
# 变更后:为满足行长限制,重新格式化
if (
    self.modelexpress_source
    and self.modelexpress_transport == "transfer_engine"
):
    return True

评论区精华

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

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

风险与影响

仅修改了代码格式,不影响任何逻辑,风险极低。

对用户无影响;对代码库的唯一影响是消除了 lint 告警,使后续 CI 检查可以通过。

关联 Issue

未识别关联 Issue

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

完整报告

参与讨论