执行摘要
修复 main 分支 lint 问题
PR 标题和 body 仅提及 "fix lint in main branch",无关联 issue 或更详细的动机描述。推测是 CI 或预提交钩子检测到代码格式问题,需要修复以保持代码风格一致。
无特殊阅读价值,但可用于了解项目的代码风格期望。
无 review 评论或讨论。
PR 标题和 body 仅提及 "fix lint in main branch",无关联 issue 或更详细的动机描述。推测是 CI 或预提交钩子检测到代码格式问题,需要修复以保持代码风格一致。
无特殊阅读价值,但可用于了解项目的代码风格期望。
无 review 评论或讨论。
python/sglang/srt/model_loader/loader.py 中,将 _transfer_via_nixl 调用从一个多行调用合并为单行;同时修复了一个 logger 字符串拼接中的多余换行。python/sglang/srt/model_executor/model_runner.py 中,将 set 推导式的多行写法压缩为单行。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 |
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 链接,后续同步到相关引用后会出现在这里。
参与讨论