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 the broken transcoding throttler #7994

Merged
merged 3 commits into from Jun 27, 2022

Conversation

nyanmisaka
Copy link
Member

@nyanmisaka nyanmisaka commented Jun 22, 2022

We still need to modify ffmpeg itself for better implementation in the future.
Or utilize the realtime filter to limit the transcoding speed.

Changes

  • Fix the broken transcoding throttler
  • Re-enable it for HWA (tested on Intel/AMD/Nvidia)

Issues
HLS and progressive StreamStates are always disposed prematurely.

@nyanmisaka nyanmisaka added the stable backport Backport into the next stable release label Jun 22, 2022
@nyanmisaka nyanmisaka force-pushed the fix-throttler branch 2 times, most recently from 6d85721 to 6b6a9f6 Compare June 22, 2022 21:45
@nyanmisaka nyanmisaka force-pushed the fix-throttler branch 2 times, most recently from 4cbf057 to d166f77 Compare June 23, 2022 08:56
@Shadowghost
Copy link
Contributor

Debug build is failing (writing it here since we missed it on a PR last time).

Copy link
Member

@cvium cvium left a comment

Choose a reason for hiding this comment

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

I see a lot of unrelated changes with the GH app. Maybe you need to revise?

@Shadowghost
Copy link
Contributor

Seems to be fine on browser: It's a +11 -26 change not touching anything unrelated as far as I can see

@nyanmisaka
Copy link
Member Author

Rebase fixes the weird issue on GH client.

@joshuaboniface joshuaboniface merged commit a7d45b5 into jellyfin:release-10.8.z Jun 27, 2022
@nyanmisaka nyanmisaka deleted the fix-throttler branch June 27, 2022 01:40
jellyfin-bot pushed a commit that referenced this pull request Jun 29, 2022
(cherry picked from commit a7d45b5)
Signed-off-by: Joshua Boniface <joshua@boniface.me>
@jellyfin-bot jellyfin-bot removed the stable backport Backport into the next stable release label Jun 29, 2022
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.

None yet

6 participants