Skip to content

Comments

Update plugin configuration: enable st22_svt_jpeg_xs#398

Merged
tszumski merged 1 commit intoOpenVisualCloud:mainfrom
tszumski:tszumski-fix-config
Jun 6, 2025
Merged

Update plugin configuration: enable st22_svt_jpeg_xs#398
tszumski merged 1 commit intoOpenVisualCloud:mainfrom
tszumski:tszumski-fix-config

Conversation

@tszumski
Copy link
Collaborator

@tszumski tszumski commented Jun 6, 2025

Update plugin configuration: enable st22_svt_jpeg_xs

Copilot AI review requested due to automatic review settings June 6, 2025 13:06
@tszumski tszumski requested review from moleksy and soopel as code owners June 6, 2025 13:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request updates the plugin configuration by enabling the st22_svt_jpeg_xs plugin and disabling the st22_sample plugin. The key changes include adding two plugin entries for st22_svt_jpeg_xs with different library paths and updating the enabled flags for the st22_sample plugin.

Comments suppressed due to low confidence (1)

media-proxy/imtl.json:10

  • The plugin 'st22_svt_jpeg_xs' is defined twice with different paths. If both entries are intended to support different environments (e.g., different architectures), consider adding an additional property (like 'platform' or 'arch') to differentiate them clearly.
"name": "st22_svt_jpeg_xs"

@ko80 ko80 requested review from Mionsz and removed request for soopel June 6, 2025 13:07
Copy link
Collaborator

@MateuszGrabuszynski MateuszGrabuszynski left a comment

Choose a reason for hiding this comment

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

lgtm

@tszumski tszumski merged commit 496e976 into OpenVisualCloud:main Jun 6, 2025
9 checks passed
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.

3 participants