Prhub

#32091 [CI] Fix failures on main

原始 PR 作者 mmangkad 合并时间 2026-07-23 00:11 文件变更 2 提交数 2 评论 5 代码增减 +4 / -3

执行摘要

修复 CI main 分支导入和测试断言问题

PR #24651 在 qwen3_5.py 中引入了 get_server_args 的调用却未添加对应导入,导致 Ruff F821 错误;PR #31312 中对 batch_size 的切片返回新的张量对象,导致使用 assertIs 的 n-gram 测试失败。这些失败阻碍了 CI 流水线的正常运行。

建议快速合并以恢复 CI 健康状态。该 PR 可作为示例:展示如何在引入新 API 后及时更新相关模块的导入,以及如何因生产代码变更而修正测试断言。

讨论亮点

无 review 评论。

实现拆解

  1. 修复导入问题:在 python/sglang/srt/models/qwen3_5.pyfrom sglang.srt.runtime_context import 列表中新增 get_server_args
  2. 修复测试断言:在 test/registered/unit/model_executor/model_runner_components/test_ngram_embedding_manager.py 中,将三处 self.assertIs(...) 替换为 self.assertTrue(torch.equal(...)),以检查张量值而非对象身份。
文件 模块 状态 重要度
python/sglang/srt/models/qwen3_5.py 模型层 modified 4.79
test/registered/unit/model_executor/model_runner_components/test_ngram_embedding_manager.py 测试 modified 4.16

关键源码片段

python/sglang/srt/models/qwen3_5.py core-logic

添加缺失的 `get_server_args` 导入,修复 Ruff F821 lint 错误。

# python/sglang/srt/models/qwen3_5.py (partial)
from sglang.srt.runtime_context import (
    get_exec,
    get_forward,
    get_parallel,
    get_server_args, # 添加此行以修复 Ruff F821: 未定义名称
    get_stream,
)
test/registered/unit/model_executor/model_runner_components/test_ngram_embedding_manager.py test-coverage

修复 n-gram 测试断言,将对象恒等检查替换为张量值相等检查,以适应生产代码切片行为。

# test/registered/unit/model_executor/model_runner_components/test_ngram_embedding_manager.py (partial)
        # 修正前:使用对象恒等检查,但由于切片返回新对象而失败
        # self.assertIs(kwargs["row_indices"], req_pool_indices)
        # 修正后:使用值相等检查
        self.assertTrue(torch.equal(kwargs["row_indices"], req_pool_indices))
        self.assertTrue(torch.equal(kwargs["column_starts"], info.out_column_starts))
        self.assertTrue(torch.equal(kwargs["req_lens"], info.out_req_lens))

评论区精华

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

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

风险与影响

风险极低:导入修复遵循已有模式,仅添加一行 import;测试断言从对象恒等改为值相等,与生产代码语义一致,不会遗漏回归。

直接影响 CI 流水线,恢复 main 分支的 lint 检查和 n-gram 单元测试的通过状态。不影响用户功能或生产行为。

关联 Issue

#32083 [CI] Fix n-gram tensor equality test

完整报告

参与讨论