Skip to content

[CI/Build] Bump PyNvVideoCodec version - #47139

Merged
WoosukKwon merged 1 commit into
vllm-project:mainfrom
Isotr0py:bump-pynvcodec
Jun 30, 2026
Merged

WoosukKwon merged 1 commit into
vllm-project:mainfrom
Isotr0py:bump-pynvcodec

Conversation

@Isotr0py

@Isotr0py Isotr0py commented Jun 30, 2026

Copy link
Copy Markdown
Member

PLEASE FILL IN THE PR DESCRIPTION HERE ENSURING ALL CHECKLIST ITEMS (AT THE BOTTOM) HAVE BEEN CONSIDERED.

Purpose

Test Plan

vllm serve Qwen/Qwen3-VL-4B-Instruct --media-io-kwargs.video.backend=pynvvideocodec
pytest -s -v tests/multimodal/test_video.py -k pynv

Test Result

$ python examples/generate/multimodal/openai_chat_completion_client_for_multimodal.py -c video
Chat completion output from video url:
 A baby wearing glasses is sitting on a bed and flipping through the pages of a book. The baby is holding the book with both hands and appears to be reading it. The baby occasionally looks up from the book and smiles.
Chat completion output from base64 encoded video:
 A baby wearing glasses is sitting on a bed and flipping through the pages of a book. The baby is holding the book with both hands and appears to be reading it. The baby occasionally looks up from the book and smiles.
tests/multimodal/test_video.py::test_pynvvideocodec_backend_accounts_raw_decoded_frames PASSED
tests/multimodal/test_video.py::test_pynvvideocodec_codec_uses_dynamic_sampling_strategy PASSED
tests/multimodal/test_video.py::test_pynvvideocodec_decoder_slots_are_bounded PASSED
tests/multimodal/test_video.py::test_pynvvideocodec_decoder_slot_retains_simple_decoder 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.

BEFORE SUBMITTING, PLEASE READ https://docs.vllm.ai/en/latest/contributing (anything written below this line will be removed by GitHub Actions)

Signed-off-by: Isotr0py <Isotr0py@outlook.com>

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

@github-project-automation github-project-automation Bot moved this to Ready in NVIDIA Jun 30, 2026
@Harry-Chen Harry-Chen added the ready ONLY add when PR is ready to merge/full CI is needed label Jun 30, 2026
@mergify

This comment was marked as resolved.

@Harry-Chen

Harry-Chen commented Jun 30, 2026

Copy link
Copy Markdown
Member

A release pipeline run is triggered at: https://buildkite.com/vllm/release-v2/builds/3338

Harry-Chen added a commit that referenced this pull request Jun 30, 2026
Signed-off-by: Shengqi Chen <harry-chen@outlook.com>
@WoosukKwon
WoosukKwon merged commit ab80b3d into vllm-project:main Jun 30, 2026
228 of 236 checks passed
@github-project-automation github-project-automation Bot moved this from Ready to Done in NVIDIA Jun 30, 2026
@brandonpelfrey

Copy link
Copy Markdown
Contributor

@Harry-Chen Just seeing this in my morning time. pynvvideocodec 2.1.0 is known to have an issue which is why we had 2.0.4 as the set dependency. (I'm asking the team to better understand if it affects us here in vLLM or not). Can you tell me which architectures/versions of the wheels are required for the vLLM pipeline? I saw your comment in the original PR that the wheels for 2.0.4 didn't support the necessary configuration, but it wasn't clear to me which version(s) you need for aarch64? We can make sure going forward that necessary versions are being built in future releases of pynvvideocodec.

@brandonpelfrey

Copy link
Copy Markdown
Contributor

@Harry-Chen I just asked and it sounds like we could publish another wheel for 2.0.4 if we can know the architectures required.

@Harry-Chen

Copy link
Copy Markdown
Member

@Harry-Chen I just asked and it sounds like we could publish another wheel for 2.0.4 if we can know the architectures required.

Yes, if you can have the manylinux_2_28_aarch64 wheel uploaded for 2.0.4, we can surely revert the dependency.

@JaheimLee

Copy link
Copy Markdown

@Harry-Chen @brandonpelfrey Hi. Why doesn't PyNvVideoCodec==2.1.0 have wheels for cp313 and above?

@Harry-Chen

Copy link
Copy Markdown
Member

@Harry-Chen @brandonpelfrey Hi. Why doesn't PyNvVideoCodec==2.1.0 have wheels for cp313 and above?

Oops. I think we still need to make it optional then.

noooop pushed a commit to noooop/vllm that referenced this pull request Jul 9, 2026
Signed-off-by: wang.yuqi <yuqi.wang@daocloud.io>
philippesic pushed a commit to philippesic/vllm-semantic-cache that referenced this pull request Jul 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants