[PR #10758] [MERGED] Add ITranscodeManager #12425

New issue

Closed

opened 2025-12-22 09:17:52 +01:00 by backuprepo · 0 comments

backuprepo commented

2025-12-22 09:17:52 +01:00

Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/10758
Author: @barronpm
Created: 12/21/2023
Status: ✅ Merged
Merged: 12/29/2023
Merged by: @Bond-009

Base: master ← Head: transcode-manager

📝 Commits (4)

abd74fd Move TranscodingJobDto and TranscodingThrottler to Controller
c208195 Rename and clean up TranscodingJob
9215a4d Add ITranscodeManager service
c49539c Move ProgressiveFileStream to Controller

📊 Changes

24 files changed (+398 additions, -455 deletions)

View changed files

📝 Emby.Server.Implementations/ApplicationHost.cs (+2 -1)
📝 Emby.Server.Implementations/LiveTv/EmbyTV/DirectRecorder.cs (+1 -0)
📝 Jellyfin.Api/Controllers/AudioController.cs (+1 -0)
📝 Jellyfin.Api/Controllers/DynamicHlsController.cs (+28 -27)
📝 Jellyfin.Api/Controllers/HlsSegmentController.cs (+7 -7)
📝 Jellyfin.Api/Controllers/LiveTvController.cs (+7 -5)
📝 Jellyfin.Api/Controllers/PlaystateController.cs (+9 -8)
📝 Jellyfin.Api/Controllers/UniversalAudioController.cs (+1 -0)
📝 Jellyfin.Api/Controllers/VideosController.cs (+8 -8)
📝 Jellyfin.Api/Helpers/AudioHelper.cs (+8 -8)
📝 Jellyfin.Api/Helpers/DynamicHlsHelper.cs (+6 -6)
📝 Jellyfin.Api/Helpers/FileStreamResponseHelpers.cs (+15 -9)
📝 Jellyfin.Api/Helpers/HlsHelpers.cs (+1 -0)
📝 Jellyfin.Api/Helpers/StreamingHelpers.cs (+5 -5)
📝 Jellyfin.Api/Models/StreamingDtos/HlsAudioRequestDto.cs (+3 -1)
📝 Jellyfin.Api/Models/StreamingDtos/HlsVideoRequestDto.cs (+3 -1)
➕ MediaBrowser.Controller/MediaEncoding/ITranscodeManager.cs (+104 -0)
📝 MediaBrowser.Controller/MediaEncoding/TranscodingJob.cs (+62 -65)
📝 MediaBrowser.Controller/MediaEncoding/TranscodingThrottler.cs (+4 -5)
📝 MediaBrowser.Controller/Streaming/ProgressiveFileStream.cs (+9 -9)

...and 4 more files

📄 Description

Changes

Moves some unrelated logic out of Jellyfin.Api
Renames and cleans up TranscodingJob
Refactors TranscodingJobHelper into ITranscodeManager service

Probably best reviewed commit-by-commit

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/10758 **Author:** [@barronpm](https://github.com/barronpm) **Created:** 12/21/2023 **Status:** ✅ Merged **Merged:** 12/29/2023 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `transcode-manager` --- ### 📝 Commits (4) - [`abd74fd`](https://github.com/jellyfin/jellyfin/commit/abd74fd5a481d67af1414960be7b7b19c9ee7e82) Move TranscodingJobDto and TranscodingThrottler to Controller - [`c208195`](https://github.com/jellyfin/jellyfin/commit/c2081955c8b2a81eb214f321697d3462709164e0) Rename and clean up TranscodingJob - [`9215a4d`](https://github.com/jellyfin/jellyfin/commit/9215a4d40ae24e5996a5e16dfa296b09a7befc40) Add ITranscodeManager service - [`c49539c`](https://github.com/jellyfin/jellyfin/commit/c49539cbe0152d23d82d7710ff2bc32e5d3d187b) Move ProgressiveFileStream to Controller ### 📊 Changes **24 files changed** (+398 additions, -455 deletions) <details> <summary>View changed files</summary> 📝 `Emby.Server.Implementations/ApplicationHost.cs` (+2 -1) 📝 `Emby.Server.Implementations/LiveTv/EmbyTV/DirectRecorder.cs` (+1 -0) 📝 `Jellyfin.Api/Controllers/AudioController.cs` (+1 -0) 📝 `Jellyfin.Api/Controllers/DynamicHlsController.cs` (+28 -27) 📝 `Jellyfin.Api/Controllers/HlsSegmentController.cs` (+7 -7) 📝 `Jellyfin.Api/Controllers/LiveTvController.cs` (+7 -5) 📝 `Jellyfin.Api/Controllers/PlaystateController.cs` (+9 -8) 📝 `Jellyfin.Api/Controllers/UniversalAudioController.cs` (+1 -0) 📝 `Jellyfin.Api/Controllers/VideosController.cs` (+8 -8) 📝 `Jellyfin.Api/Helpers/AudioHelper.cs` (+8 -8) 📝 `Jellyfin.Api/Helpers/DynamicHlsHelper.cs` (+6 -6) 📝 `Jellyfin.Api/Helpers/FileStreamResponseHelpers.cs` (+15 -9) 📝 `Jellyfin.Api/Helpers/HlsHelpers.cs` (+1 -0) 📝 `Jellyfin.Api/Helpers/StreamingHelpers.cs` (+5 -5) 📝 `Jellyfin.Api/Models/StreamingDtos/HlsAudioRequestDto.cs` (+3 -1) 📝 `Jellyfin.Api/Models/StreamingDtos/HlsVideoRequestDto.cs` (+3 -1) ➕ `MediaBrowser.Controller/MediaEncoding/ITranscodeManager.cs` (+104 -0) 📝 `MediaBrowser.Controller/MediaEncoding/TranscodingJob.cs` (+62 -65) 📝 `MediaBrowser.Controller/MediaEncoding/TranscodingThrottler.cs` (+4 -5) 📝 `MediaBrowser.Controller/Streaming/ProgressiveFileStream.cs` (+9 -9) _...and 4 more files_ </details> ### 📄 Description **Changes** - Moves some unrelated logic out of `Jellyfin.Api` - Renames and cleans up `TranscodingJob` - Refactors `TranscodingJobHelper` into `ITranscodeManager` service Probably best reviewed commit-by-commit --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

backuprepo

2025-12-22 09:17:52 +01:00

closed this issue
added the
pull-request
label

backuprepo referenced this issue

2025-12-22 09:37:07 +01:00

[PR #12425] [MERGED] Don't force non-virtual when all episodes in season are isMissing=true #13011

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: starred/jellyfin#12425

No description provided.

Rows
Columns