Skip to content

[Misc] Move PyNvVideoCodec stuff out of gpu worker - #49322

Merged
noooop merged 1 commit into
vllm-project:mainfrom
Isotr0py:clean-nvcodec
Jul 22, 2026
Merged

[Misc] Move PyNvVideoCodec stuff out of gpu worker#49322
noooop merged 1 commit into
vllm-project:mainfrom
Isotr0py:clean-nvcodec

Conversation

@Isotr0py

@Isotr0py Isotr0py commented Jul 21, 2026 •
edited by github-actions Bot
Loading

Copy link
Copy Markdown
Member

Purpose

Test Plan

Test Result

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.

@Isotr0py
move pynvvideocodec out of gpu worker
5fcd6a5 
Signed-off-by: Isotr0py <Isotr0py@outlook.com>
claude[bot]
claude Bot reviewed Jul 21, 2026
View reviewed changes

@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.

@mergify mergify Bot added multi-modality Related to multi-modality (#4194) v1 labels Jul 21, 2026
@Isotr0py Isotr0py added the ready ONLY add when PR is ready to merge/full CI is needed label Jul 21, 2026
@noooop
noooop merged commit 1750e44 into vllm-project:main Jul 22, 2026
118 of 122 checks passed
@Isotr0py
Isotr0py deleted the clean-nvcodec branch July 22, 2026 02:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@claude claude[bot] claude[bot] left review comments
@noooop noooop noooop approved these changes
@DarkLight1337 DarkLight1337 Awaiting requested review from DarkLight1337 DarkLight1337 is a code owner
@ywang96 ywang96 Awaiting requested review from ywang96 ywang96 is a code owner
@NickLucche NickLucche Awaiting requested review from NickLucche NickLucche is a code owner
@njhill njhill Awaiting requested review from njhill njhill is a code owner
@WoosukKwon WoosukKwon Awaiting requested review from WoosukKwon WoosukKwon is a code owner
@youkaichao youkaichao Awaiting requested review from youkaichao youkaichao is a code owner
@robertgshaw2-redhat robertgshaw2-redhat Awaiting requested review from robertgshaw2-redhat robertgshaw2-redhat is a code owner
@mgoin mgoin Awaiting requested review from mgoin mgoin is a code owner
@tlrmchlsmth tlrmchlsmth Awaiting requested review from tlrmchlsmth tlrmchlsmth is a code owner
@houseroad houseroad Awaiting requested review from houseroad houseroad is a code owner
@yewentao256 yewentao256 Awaiting requested review from yewentao256 yewentao256 is a code owner
@ProExpertProg ProExpertProg Awaiting requested review from ProExpertProg ProExpertProg is a code owner
@tjtanaa tjtanaa Awaiting requested review from tjtanaa tjtanaa is a code owner

Assignees

No one assigned

Labels

multi-modality Related to multi-modality (#4194) ready ONLY add when PR is ready to merge/full CI is needed v1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Isotr0py @noooop