test_utils.py 14.6 KB
Newer Older
1
# SPDX-License-Identifier: Apache-2.0
2
# SPDX-FileCopyrightText: Copyright contributors to the vLLM project
3
# ruff: noqa
4

5
import json
6
7
8
import os
import tempfile
from pathlib import Path
9
from unittest.mock import patch
10

11
import pytest
12
import torch
13
import yaml
14
from transformers import AutoTokenizer
15

16
from vllm.config import ParallelConfig, VllmConfig, set_current_vllm_config
17
from vllm.transformers_utils.detokenizer_utils import convert_ids_list_to_tokens
18

19
from vllm.utils import (
20
21
22
23
    FlexibleArgumentParser,
    bind_kv_cache,
    unique_filepath,
)
24
from ..utils import create_new_process_for_each_test, flat_product
25

26

27
28
29
30
# Tests for FlexibleArgumentParser
@pytest.fixture
def parser():
    parser = FlexibleArgumentParser()
31
32
33
34
35
36
37
38
    parser.add_argument(
        "--image-input-type", choices=["pixel_values", "image_features"]
    )
    parser.add_argument("--model-name")
    parser.add_argument("--batch-size", type=int)
    parser.add_argument("--enable-feature", action="store_true")
    parser.add_argument("--hf-overrides", type=json.loads)
    parser.add_argument("-O", "--compilation-config", type=json.loads)
39
40
41
    return parser


42
43
44
@pytest.fixture
def parser_with_config():
    parser = FlexibleArgumentParser()
45
46
47
48
49
50
51
52
    parser.add_argument("serve")
    parser.add_argument("model_tag", nargs="?")
    parser.add_argument("--model", type=str)
    parser.add_argument("--served-model-name", type=str)
    parser.add_argument("--config", type=str)
    parser.add_argument("--port", type=int)
    parser.add_argument("--tensor-parallel-size", type=int)
    parser.add_argument("--trust-remote-code", action="store_true")
53
54
55
    return parser


56
def test_underscore_to_dash(parser):
57
58
    args = parser.parse_args(["--image_input_type", "pixel_values"])
    assert args.image_input_type == "pixel_values"
59
60
61


def test_mixed_usage(parser):
62
63
64
65
66
    args = parser.parse_args(
        ["--image_input_type", "image_features", "--model-name", "facebook/opt-125m"]
    )
    assert args.image_input_type == "image_features"
    assert args.model_name == "facebook/opt-125m"
67
68
69
70


def test_with_equals_sign(parser):
    args = parser.parse_args(
71
72
73
74
        ["--image_input_type=pixel_values", "--model-name=facebook/opt-125m"]
    )
    assert args.image_input_type == "pixel_values"
    assert args.model_name == "facebook/opt-125m"
75
76
77


def test_with_int_value(parser):
78
    args = parser.parse_args(["--batch_size", "32"])
79
    assert args.batch_size == 32
80
    args = parser.parse_args(["--batch-size", "32"])
81
82
83
84
    assert args.batch_size == 32


def test_with_bool_flag(parser):
85
    args = parser.parse_args(["--enable_feature"])
86
    assert args.enable_feature is True
87
    args = parser.parse_args(["--enable-feature"])
88
89
90
91
92
    assert args.enable_feature is True


def test_invalid_choice(parser):
    with pytest.raises(SystemExit):
93
        parser.parse_args(["--image_input_type", "invalid_choice"])
94
95
96


def test_missing_required_argument(parser):
97
    parser.add_argument("--required-arg", required=True)
98
99
    with pytest.raises(SystemExit):
        parser.parse_args([])
100
101


102
def test_cli_override_to_config(parser_with_config, cli_config_file):
103
104
105
    args = parser_with_config.parse_args(
        ["serve", "mymodel", "--config", cli_config_file, "--tensor-parallel-size", "3"]
    )
106
    assert args.tensor_parallel_size == 3
107
108
109
    args = parser_with_config.parse_args(
        ["serve", "mymodel", "--tensor-parallel-size", "3", "--config", cli_config_file]
    )
110
    assert args.tensor_parallel_size == 3
111
    assert args.port == 12312
112
113
114
115
116
117
118
119
120
121
122
123
    args = parser_with_config.parse_args(
        [
            "serve",
            "mymodel",
            "--tensor-parallel-size",
            "3",
            "--config",
            cli_config_file,
            "--port",
            "666",
        ]
    )
124
125
    assert args.tensor_parallel_size == 3
    assert args.port == 666
126
127


128
def test_config_args(parser_with_config, cli_config_file):
129
    args = parser_with_config.parse_args(
130
131
        ["serve", "mymodel", "--config", cli_config_file]
    )
132
    assert args.tensor_parallel_size == 2
133
    assert args.trust_remote_code
134
135
136
137


def test_config_file(parser_with_config):
    with pytest.raises(FileNotFoundError):
138
        parser_with_config.parse_args(
139
140
            ["serve", "mymodel", "--config", "test_config.yml"]
        )
141
142
143

    with pytest.raises(ValueError):
        parser_with_config.parse_args(
144
145
            ["serve", "mymodel", "--config", "./data/test_config.json"]
        )
146
147

    with pytest.raises(ValueError):
148
149
150
151
152
153
154
155
156
157
158
        parser_with_config.parse_args(
            [
                "serve",
                "mymodel",
                "--tensor-parallel-size",
                "3",
                "--config",
                "--batch-size",
                "32",
            ]
        )
159
160


161
def test_no_model_tag(parser_with_config, cli_config_file):
162
    with pytest.raises(ValueError):
163
        parser_with_config.parse_args(["serve", "--config", cli_config_file])
164
165


166
167
168
169
170
def test_dict_args(parser):
    args = [
        "--model-name=something.something",
        "--hf-overrides.key1",
        "val1",
171
        # Test nesting
172
173
174
175
        "--hf-overrides.key2.key3",
        "val2",
        "--hf-overrides.key2.key4",
        "val3",
176
        # Test compile config and compilation mode
177
178
179
180
        "-O.use_inductor=true",
        "-O.backend",
        "custom",
        "-O1",
181
        # Test = sign
182
        "--hf-overrides.key5=val4",
183
184
185
186
187
        # Test underscore to dash conversion
        "--hf_overrides.key_6",
        "val5",
        "--hf_overrides.key-7.key_8",
        "val6",
188
189
190
191
192
193
194
195
196
        # Test data type detection
        "--hf_overrides.key9",
        "100",
        "--hf_overrides.key10",
        "100.0",
        "--hf_overrides.key11",
        "true",
        "--hf_overrides.key12.key13",
        "null",
197
198
199
200
201
202
203
        # Test '-' and '.' in value
        "--hf_overrides.key14.key15",
        "-minus.and.dot",
        # Test array values
        "-O.custom_ops+",
        "-quant_fp8",
        "-O.custom_ops+=+silu_mul,-rms_norm",
204
205
206
207
208
209
210
211
212
213
    ]
    parsed_args = parser.parse_args(args)
    assert parsed_args.model_name == "something.something"
    assert parsed_args.hf_overrides == {
        "key1": "val1",
        "key2": {
            "key3": "val2",
            "key4": "val3",
        },
        "key5": "val4",
214
215
216
217
        "key_6": "val5",
        "key-7": {
            "key_8": "val6",
        },
218
219
220
221
222
223
        "key9": 100,
        "key10": 100.0,
        "key11": True,
        "key12": {
            "key13": None,
        },
224
225
        "key14": {
            "key15": "-minus.and.dot",
226
        },
227
    }
228
    assert parsed_args.compilation_config == {
229
        "mode": 1,
230
231
232
233
234
235
236
237
238
239
240
241
242
243
        "use_inductor": True,
        "backend": "custom",
        "custom_ops": ["-quant_fp8", "+silu_mul", "-rms_norm"],
    }


def test_duplicate_dict_args(caplog_vllm, parser):
    args = [
        "--model-name=something.something",
        "--hf-overrides.key1",
        "val1",
        "--hf-overrides.key1",
        "val2",
        "-O1",
244
        "-O.mode",
245
246
247
248
249
250
251
        "2",
        "-O3",
    ]

    parsed_args = parser.parse_args(args)
    # Should be the last value
    assert parsed_args.hf_overrides == {"key1": "val2"}
252
    assert parsed_args.compilation_config == {"mode": 3}
253
254
255
256

    assert len(caplog_vllm.records) == 1
    assert "duplicate" in caplog_vllm.text
    assert "--hf-overrides.key1" in caplog_vllm.text
257
    assert "-O.mode" in caplog_vllm.text
258
259


260
261
262
263
def test_bind_kv_cache():
    from vllm.attention import Attention

    ctx = {
264
265
266
267
        "layers.0.self_attn": Attention(32, 128, 0.1),
        "layers.1.self_attn": Attention(32, 128, 0.1),
        "layers.2.self_attn": Attention(32, 128, 0.1),
        "layers.3.self_attn": Attention(32, 128, 0.1),
268
269
    }
    kv_cache = [
270
271
272
273
        torch.zeros((1,)),
        torch.zeros((1,)),
        torch.zeros((1,)),
        torch.zeros((1,)),
274
275
    ]
    bind_kv_cache(ctx, [kv_cache])
276
277
278
279
280
    assert ctx["layers.0.self_attn"].kv_cache[0] is kv_cache[0]
    assert ctx["layers.1.self_attn"].kv_cache[0] is kv_cache[1]
    assert ctx["layers.2.self_attn"].kv_cache[0] is kv_cache[2]
    assert ctx["layers.3.self_attn"].kv_cache[0] is kv_cache[3]

281

282
283
284
285
def test_bind_kv_cache_kv_sharing():
    from vllm.attention import Attention

    ctx = {
286
287
288
289
        "layers.0.self_attn": Attention(32, 128, 0.1),
        "layers.1.self_attn": Attention(32, 128, 0.1),
        "layers.2.self_attn": Attention(32, 128, 0.1),
        "layers.3.self_attn": Attention(32, 128, 0.1),
290
291
    }
    kv_cache = [
292
293
294
295
        torch.zeros((1,)),
        torch.zeros((1,)),
        torch.zeros((1,)),
        torch.zeros((1,)),
296
297
    ]
    shared_kv_cache_layers = {
298
299
        "layers.2.self_attn": "layers.1.self_attn",
        "layers.3.self_attn": "layers.0.self_attn",
300
301
    }
    bind_kv_cache(ctx, [kv_cache], shared_kv_cache_layers)
302
303
304
305
306
    assert ctx["layers.0.self_attn"].kv_cache[0] is kv_cache[0]
    assert ctx["layers.1.self_attn"].kv_cache[0] is kv_cache[1]
    assert ctx["layers.2.self_attn"].kv_cache[0] is kv_cache[1]
    assert ctx["layers.3.self_attn"].kv_cache[0] is kv_cache[0]

307

308
309
310
311
312
def test_bind_kv_cache_non_attention():
    from vllm.attention import Attention

    # example from Jamba PP=2
    ctx = {
313
314
        "model.layers.20.attn": Attention(32, 128, 0.1),
        "model.layers.28.attn": Attention(32, 128, 0.1),
315
316
    }
    kv_cache = [
317
318
        torch.zeros((1,)),
        torch.zeros((1,)),
319
320
    ]
    bind_kv_cache(ctx, [kv_cache])
321
322
    assert ctx["model.layers.20.attn"].kv_cache[0] is kv_cache[0]
    assert ctx["model.layers.28.attn"].kv_cache[0] is kv_cache[1]
323
324
325


def test_bind_kv_cache_pp():
326
    with patch("vllm.utils.torch_utils.cuda_device_count_stateless", lambda: 2):
327
        # this test runs with 1 GPU, but we simulate 2 GPUs
328
        cfg = VllmConfig(parallel_config=ParallelConfig(pipeline_parallel_size=2))
329
330
331
332
    with set_current_vllm_config(cfg):
        from vllm.attention import Attention

        ctx = {
333
            "layers.0.self_attn": Attention(32, 128, 0.1),
334
        }
335
        kv_cache = [[torch.zeros((1,))], [torch.zeros((1,))]]
336
        bind_kv_cache(ctx, kv_cache)
337
338
        assert ctx["layers.0.self_attn"].kv_cache[0] is kv_cache[0][0]
        assert ctx["layers.0.self_attn"].kv_cache[1] is kv_cache[1][0]
339
340


341
342
343
def test_model_specification(
    parser_with_config, cli_config_file, cli_config_file_with_model
):
344
    # Test model in CLI takes precedence over config
345
    args = parser_with_config.parse_args(
346
347
348
349
        ["serve", "cli-model", "--config", cli_config_file_with_model]
    )
    assert args.model_tag == "cli-model"
    assert args.served_model_name == "mymodel"
350
351

    # Test model from config file works
352
353
354
355
356
357
358
359
360
    args = parser_with_config.parse_args(
        [
            "serve",
            "--config",
            cli_config_file_with_model,
        ]
    )
    assert args.model == "config-model"
    assert args.served_model_name == "mymodel"
361
362
363

    # Test no model specified anywhere raises error
    with pytest.raises(ValueError, match="No model specified!"):
364
        parser_with_config.parse_args(["serve", "--config", cli_config_file])
365
366

    # Test using --model option raises error
367
368
369
370
371
372
373
374
375
376
377
    # with pytest.raises(
    #         ValueError,
    #         match=
    #     ("With `vllm serve`, you should provide the model as a positional "
    #      "argument or in a config file instead of via the `--model` option."),
    # ):
    #     parser_with_config.parse_args(['serve', '--model', 'my-model'])

    # Test using --model option back-compatibility
    # (when back-compatibility ends, the above test should be uncommented
    # and the below test should be removed)
378
379
380
381
382
383
384
385
386
387
388
389
    args = parser_with_config.parse_args(
        [
            "serve",
            "--tensor-parallel-size",
            "2",
            "--model",
            "my-model",
            "--trust-remote-code",
            "--port",
            "8001",
        ]
    )
390
391
392
393
394
    assert args.model is None
    assert args.tensor_parallel_size == 2
    assert args.trust_remote_code is True
    assert args.port == 8001

395
396
397
398
399
400
401
402
403
    args = parser_with_config.parse_args(
        [
            "serve",
            "--tensor-parallel-size=2",
            "--model=my-model",
            "--trust-remote-code",
            "--port=8001",
        ]
    )
404
405
406
407
    assert args.model is None
    assert args.tensor_parallel_size == 2
    assert args.trust_remote_code is True
    assert args.port == 8001
408
409

    # Test other config values are preserved
410
411
412
413
414
415
416
417
    args = parser_with_config.parse_args(
        [
            "serve",
            "cli-model",
            "--config",
            cli_config_file_with_model,
        ]
    )
418
419
420
421
422
    assert args.tensor_parallel_size == 2
    assert args.trust_remote_code is True
    assert args.port == 12312


423
424
425
426
def test_convert_ids_list_to_tokens():
    tokenizer = AutoTokenizer.from_pretrained("Qwen/Qwen2.5-1.5B-Instruct")
    token_ids = tokenizer.encode("Hello, world!")
    # token_ids = [9707, 11, 1879, 0]
427
    assert tokenizer.convert_ids_to_tokens(token_ids) == ["Hello", ",", "Ġworld", "!"]
428
    tokens = convert_ids_list_to_tokens(tokenizer, token_ids)
429
    assert tokens == ["Hello", ",", " world", "!"]
430
431


432
433
434
435
436
437
def test_load_config_file(tmp_path):
    # Define the configuration data
    config_data = {
        "enable-logging": True,
        "list-arg": ["item1", "item2"],
        "port": 12323,
438
        "tensor-parallel-size": 4,
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
    }

    # Write the configuration data to a temporary YAML file
    config_file_path = tmp_path / "config.yaml"
    with open(config_file_path, "w") as config_file:
        yaml.dump(config_data, config_file)

    # Initialize the parser
    parser = FlexibleArgumentParser()

    # Call the function with the temporary file path
    processed_args = parser.load_config_file(str(config_file_path))

    # Expected output
    expected_args = [
        "--enable-logging",
        "--list-arg",
        "item1",
        "item2",
        "--port",
        "12323",
        "--tensor-parallel-size",
        "4",
    ]

    # Assert that the processed arguments match the expected output
    assert processed_args == expected_args
    os.remove(str(config_file_path))
467
468
469
470
471
472
473
474
475
476
477
478


def test_unique_filepath():
    temp_dir = tempfile.mkdtemp()
    path_fn = lambda i: Path(temp_dir) / f"file_{i}.txt"
    paths = set()
    for i in range(10):
        path = unique_filepath(path_fn)
        path.write_text("test")
        paths.add(path)
    assert len(paths) == 10
    assert len(list(Path(temp_dir).glob("*.txt"))) == 10
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500


def test_flat_product():
    # Check regular itertools.product behavior
    result1 = list(flat_product([1, 2, 3], ["a", "b"]))
    assert result1 == [
        (1, "a"),
        (1, "b"),
        (2, "a"),
        (2, "b"),
        (3, "a"),
        (3, "b"),
    ]

    # check that the tuples get flattened
    result2 = list(flat_product([(1, 2), (3, 4)], ["a", "b"], [(5, 6)]))
    assert result2 == [
        (1, 2, "a", 5, 6),
        (1, 2, "b", 5, 6),
        (3, 4, "a", 5, 6),
        (3, 4, "b", 5, 6),
    ]