From c4b37471c412fff7217abf1693e9d1e2d8f6d8f0 Mon Sep 17 00:00:00 2001 From: tzhouam Date: Thu, 10 Sep 2026 07:58:27 +0800 Subject: [PATCH] test(qwen3_tts): accept the mapper argument in the code2wav weight-loader double MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Nightly 3034 (Simple ยท Model Executor Test) failed 11 cases in tests/model_executor/models/qwen3_tts/test_qwen3_tts_code2wav.py with TypeError: _FakeAutoWeightsLoader.load_weights() got an unexpected keyword argument 'mapper' vLLM 0.29 removed skip_prefixes from AutoWeightsLoader, so on this branch Qwen3TTSCode2Wav.load_weights passes the "encoder." exclusion as a WeightsMapper via load_weights(weights, mapper=...). The test double still modelled the old one-argument signature. Let it accept the keyword. Branch-only rebase fallout; no production code changed. Validation: tests/model_executor/models/qwen3_tts passes (232 tests). Co-Authored-By: Claude Fable 5.1 Claude-Session: https://claude.ai/code/session_01MpXe2LeRrUQGjZfKFaocMZ Signed-off-by: tzhouam --- .../models/qwen3_tts/test_qwen3_tts_code2wav.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/tests/model_executor/models/qwen3_tts/test_qwen3_tts_code2wav.py b/tests/model_executor/models/qwen3_tts/test_qwen3_tts_code2wav.py index 45b40d026..64f9ac8de 100644 --- a/tests/model_executor/models/qwen3_tts/test_qwen3_tts_code2wav.py +++ b/tests/model_executor/models/qwen3_tts/test_qwen3_tts_code2wav.py @@ -1,5 +1,5 @@ # SPDX-License-Identifier: Apache-2.0 -# SPDX-FileCopyrightText: Copyright contributors to the vLLM project +# SPDX-FileCopyrightText: Copyright contributors to the vLLM-Omni project from types import SimpleNamespace from unittest.mock import patch @@ -157,7 +157,10 @@ def _load_weights_noop(model: Qwen3TTSCode2Wav) -> set[str]: def __init__(self, *_: object, **__: object): pass - def load_weights(self, _weights: object) -> set[str]: + def load_weights(self, _weights: object, *, mapper: object | None = None) -> set[str]: + # vLLM 0.29 dropped skip_prefixes from AutoWeightsLoader; the model + # now passes the stage exclusions as a WeightsMapper instead. + del mapper return {"decoder.fake_weight"} with (