Skip to content

Conversation

@charlotte12l
Copy link
Contributor

@charlotte12l charlotte12l commented Sep 10, 2025

Purpose

Per discussion #24277 (comment), added tests/transformers_utils directory to "Async Engine, Inputs, Utils, Worker Test"

Test Plan

CI

Test Result

In Async Engine, Inputs, Utils, Worker Test

  | [2025-09-11T08:21:19Z] + pytest -v -s transformers_utils
  | [2025-09-11T08:21:26Z] INFO 09-11 01:21:26 [__init__.py:216] Automatically detected platform cuda.
  | [2025-09-11T08:21:30Z] /usr/local/lib/python3.12/dist-packages/pytest_asyncio/plugin.py:208: PytestDeprecationWarning: The configuration option "asyncio_default_fixture_loop_scope" is unset.
  | [2025-09-11T08:21:30Z] The event loop scope for asynchronous fixtures will default to the fixture caching scope. Future versions of pytest-asyncio will default the loop scope for asynchronous fixtures to function scope. Set the default fixture loop scope explicitly in order to avoid unexpected behavior in the future. Valid fixture loop scopes are: "function", "class", "module", "package", "session"
  | [2025-09-11T08:21:30Z]
  | [2025-09-11T08:21:30Z]   warnings.warn(PytestDeprecationWarning(_DEFAULT_FIXTURE_LOOP_SCOPE_UNSET))
  | [2025-09-11T08:21:30Z] ============================= test session starts ==============================
  | [2025-09-11T08:21:30Z] platform linux -- Python 3.12.11, pytest-8.3.5, pluggy-1.5.0 -- /usr/bin/python3
  | [2025-09-11T08:21:30Z] cachedir: .pytest_cache
  | [2025-09-11T08:21:30Z] hypothesis profile 'ci' -> database=None, deadline=None, print_blob=True, derandomize=True, suppress_health_check=[HealthCheck.too_slow]
  | [2025-09-11T08:21:30Z] rootdir: /vllm-workspace
  | [2025-09-11T08:21:30Z] configfile: pyproject.toml
  | [2025-09-11T08:21:30Z] plugins: anyio-4.6.2.post1, buildkite-test-collector-0.1.9, hydra-core-1.3.2, hypothesis-6.131.0, asyncio-0.24.0, forked-1.6.0, mock-3.14.0, rerunfailures-14.0, shard-0.1.2, subtests-0.14.1, timeout-2.3.1, schemathesis-3.39.15
  | [2025-09-11T08:21:30Z] asyncio: mode=Mode.STRICT, default_loop_scope=None
  | [2025-09-11T08:21:30Z] collecting ... INFO 09-11 01:21:30 [config.py:255] Registered config parser `<class 'tests.transformers_utils.test_config_parser_registry.CustomConfigParser'>` with config format `custom_config_parser`
  | collected 2 items
  | [2025-09-11T08:21:30Z] Running 2 items in this shard: tests/transformers_utils/test_config_parser_registry.py::test_register_config_parser, tests/transformers_utils/test_config_parser_registry.py::test_invalid_config_parser
  | [2025-09-11T08:21:30Z]
  | [2025-09-11T08:21:31Z] transformers_utils/test_config_parser_registry.py::test_register_config_parser PASSED
  | [2025-09-11T08:21:31Z] transformers_utils/test_config_parser_registry.py::test_invalid_config_parser PASSED



Essential Elements of an Effective PR Description Checklist
  • The purpose of the PR, such as "Fix some issue (link existing issues this PR will resolve)".
  • The test plan, such as providing test command.
  • The test results, such as pasting the results comparison before and after, or e2e results
  • (Optional) The necessary documentation update, such as updating supported_models.md and examples for a new model.
  • (Optional) Release notes update. If your change is user facing, please update the release notes draft in the Google Doc.

Signed-off-by: Xingyu Liu <charlotteliu12x@gmail.com>
@mergify mergify bot added the ci/build label Sep 10, 2025
@charlotte12l charlotte12l changed the title Add transformers_utils to Async Engine, Inputs, Utils, Worker Test [CI]Add transformers_utils to Async Engine, Inputs, Utils, Worker Test Sep 10, 2025
@charlotte12l charlotte12l marked this pull request as ready for review September 11, 2025 06:15
@22quinn 22quinn self-requested a review September 11, 2025 07:43
@22quinn 22quinn added the ready ONLY add when PR is ready to merge/full CI is needed label Sep 11, 2025
@vllm-bot vllm-bot merged commit 0fc3646 into vllm-project:main Sep 11, 2025
76 of 86 checks passed
skyloevil pushed a commit to skyloevil/vllm that referenced this pull request Sep 13, 2025
dsxsteven pushed a commit to dsxsteven/vllm_splitPR that referenced this pull request Sep 15, 2025
xuechendi pushed a commit to vllm-project/vllm-gaudi that referenced this pull request Sep 17, 2025
vllm-project/vllm#24795 and
vllm-project/vllm#24615 and
vllm-project/vllm#24078

---------

Signed-off-by: Agata Dobrzyniewicz <adobrzyniewicz@habana.ai>
slokesha pushed a commit to slokesha/vllm-gaudi that referenced this pull request Sep 24, 2025
vllm-project/vllm#24795 and
vllm-project/vllm#24615 and
vllm-project/vllm#24078

---------

Signed-off-by: Agata Dobrzyniewicz <adobrzyniewicz@habana.ai>
Signed-off-by: slokesha <slokeshappa@habana.ai>
FeiDaLI pushed a commit to FeiDaLI/vllm that referenced this pull request Sep 25, 2025
xuebwang-amd pushed a commit to xuebwang-amd/vllm that referenced this pull request Oct 10, 2025
vllm-project#24615)

Signed-off-by: Xingyu Liu <charlotteliu12x@gmail.com>
Signed-off-by: xuebwang-amd <xuebwang@amd.com>
xuebwang-amd pushed a commit to xuebwang-amd/vllm that referenced this pull request Oct 24, 2025
vllm-project#24615)

Signed-off-by: Xingyu Liu <charlotteliu12x@gmail.com>
Signed-off-by: xuebwang-amd <xuebwang@amd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/build ready ONLY add when PR is ready to merge/full CI is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants