Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix broken hardware encoder and filter for trickplay #11506

Merged
merged 1 commit into from
May 7, 2024

Conversation

gnattu
Copy link
Member

@gnattu gnattu commented May 7, 2024

The hardware processing pipeline never worked for trickplay, because the required encoder was not added to the validator and the software encoder is always passed to the ProcessingFilter builder, which returns the software filter+encoder.

Changes

Issues

The hardware processing pipeline never worked for trickplay, because the required encoder was not added to the validator and the software encoder is always passed to the ProcessingFilter builder, which always returns the software filter+encoder.

Signed-off-by: gnattu <gnattuoc@me.com>
@gnattu gnattu added bug Something isn't working release critical A bug severe enough or impactful enough to block stable release labels May 7, 2024
@gnattu gnattu requested review from nyanmisaka and a team May 7, 2024 07:56
Copy link
Contributor

@Shadowghost Shadowghost left a comment

Choose a reason for hiding this comment

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

Shouldn't we also add mjpeg_videotoolbox and mjpeg_rkmpp?

@nyanmisaka
Copy link
Member

Shouldn't we also add mjpeg_videotoolbox and mjpeg_rkmpp?

mjpeg_rkmpp uses slightly different filter options than h26x_rkmpp, which requires more changes. I'll leave it to JF 10.10.

@gnattu
Copy link
Member Author

gnattu commented May 7, 2024

Shouldn't we also add mjpeg_videotoolbox and mjpeg_rkmpp?

These additional platforms should go with 10.10 in my opinion, they will require additional code to work.

@nielsvanvelzen nielsvanvelzen added this to In progress in Release 10.9.z via automation May 7, 2024
@nielsvanvelzen nielsvanvelzen removed the release critical A bug severe enough or impactful enough to block stable release label May 7, 2024
@crobibero crobibero merged commit 4044431 into master May 7, 2024
17 checks passed
Release 10.9.z automation moved this from In progress to Done May 7, 2024
@crobibero crobibero deleted the fix-trickplay-hardware-support branch May 7, 2024 13:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Development

Successfully merging this pull request may close these issues.

None yet

6 participants