Skip to content

DNI: DEBUG: PR to debug/fix CPU CI failures - #400

Draft
ksheth-dev wants to merge 2 commits into
dev/ksheth/pynvvideocodec_v2from
pull-request/999999
Draft

DNI: DEBUG: PR to debug/fix CPU CI failures#400
ksheth-dev wants to merge 2 commits into
dev/ksheth/pynvvideocodec_v2from
pull-request/999999

Conversation

@ksheth-dev

Copy link
Copy Markdown
Collaborator

No description provided.

Split PyNvHardwareEncoder and its ABGR-frame / NAL-scan helpers out of
encoders.py into a sibling nvenc.py that owns the top-level
PyNvVideoCodec import. encoders.py no longer touches PyNvVideoCodec at
module load; select_encoder probes availability via importlib.util.find_spec
(no side effects) and imports nvenc only when a hardware backend is
actually about to be constructed.

The Lingbot WebRTC path resolves to DefaultRTCEncoder in the base
manager and never enters the hardware branch, so its process no longer
loads PyNvVideoCodec at all — sidestepping a silent early-exit observed
during the Lingbot server launch when PyNvVideoCodec was importable.

Tests updated to patch _pynvvideocodec_installed and inject a fake
PyNvVideoCodec into sys.modules so nvenc's top-level import binds to
the mock. The GPU smoke test's imports follow the split.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@ksheth-dev
ksheth-dev force-pushed the pull-request/999999 branch from 3ce03df to b65f4c8 Compare July 29, 2026 09:09
@ksheth-dev
ksheth-dev force-pushed the dev/ksheth/pynvvideocodec_v2 branch 4 times, most recently from bec5ea2 to 6434238 Compare July 29, 2026 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant