Skip to content

Avoid creating full encoder for jetson probing capability - #1526

Draft
alan-george-lk wants to merge 3 commits into
mainfrom
alan/bugfix-jetson-capability
Draft

alan-george-lk wants to merge 3 commits into
mainfrom
alan/bugfix-jetson-capability

Conversation

@alan-george-lk

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

No changeset found

This PR modifies versioned packages but doesn't include a changeset. The following packages require a version bump:

  • libwebrtc
  • livekit
  • livekit-capture
  • livekit-ffi
  • webrtc-sys

A package must be bumped when its own files change, and whenever a package it depends on is bumped (so downstream consumers get a matching release).

Click here to create a changeset for the missing packages

The link pre-populates a changeset file with patch bumps for the missing packages. You can also add them to your existing changeset. Edit the bump types as needed before committing.

If this change doesn't require a version bump, add the internal label to this PR.

}
}
return true;
// Do not create a temporary NvVideoEncoder here. NvVideoEncoder uses

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

if we dont create it here, where do we create the encoder?

Also have we tested this on a non jetson platform? im curious why this was ever done in the first place

@alan-george-lk
alan-george-lk force-pushed the alan/bugfix-jetson-capability branch from fcc771b to 4ba7498 Compare October 16, 2026 19:31
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.

2 participants