[PR #12270] [MERGED] Add software tonemap filter support #13123

Closed
opened 2026-02-07 07:07:33 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/12270
Author: @gnattu
Created: 7/14/2024
Status: Merged
Merged: 7/15/2024
Merged by: @crobibero

Base: masterHead: sw-tonemap


📝 Commits (4)

  • 016dc9d Add SIMD optimized software tonemap support
  • 1f32f95 Add tonemapx to filter list
  • 2dee5b8 Correctly specify format for tonemap
  • 518c166 Prefer tonemapx during HDR image extraction

📊 Changes

3 files changed (+53 additions, -10 deletions)

View changed files

📝 MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs (+40 -4)
📝 MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs (+1 -0)
📝 MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs (+12 -6)

📄 Description

This adds a fast SIMD optimized software tonemap filter support to the server. This filter currently does not support dolby vision reshaping and only support HDR10 and HLG inputs. With current veryfast libx264 settings we can expect 4K60 transcoding+tone mapping on a modern processor with 8 cores equivalent to Zen3/Golden Cove/Firestorm level performance.

Changes

  • Add tonemapx filter support to software transcode pipeline
  • Add tonemapx filter support to thumbnail generation pipeline

Issues

Depends on ffmpeg pr https://github.com/jellyfin/jellyfin-ffmpeg/pull/407

The perf figures in the original post only reflects the initial implementation. The finalized perf figures are here: https://github.com/jellyfin/jellyfin-ffmpeg/pull/407#issuecomment-2198303178


🔄 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/12270 **Author:** [@gnattu](https://github.com/gnattu) **Created:** 7/14/2024 **Status:** ✅ Merged **Merged:** 7/15/2024 **Merged by:** [@crobibero](https://github.com/crobibero) **Base:** `master` ← **Head:** `sw-tonemap` --- ### 📝 Commits (4) - [`016dc9d`](https://github.com/jellyfin/jellyfin/commit/016dc9d86c80c8b9542aff650302ab35d25c84fd) Add SIMD optimized software tonemap support - [`1f32f95`](https://github.com/jellyfin/jellyfin/commit/1f32f95b9c99199657055d230a951f9be56ef86a) Add tonemapx to filter list - [`2dee5b8`](https://github.com/jellyfin/jellyfin/commit/2dee5b8d0426b51ad0c997e1b8a2d2cd20832245) Correctly specify format for tonemap - [`518c166`](https://github.com/jellyfin/jellyfin/commit/518c166a3921fb7c398522680e56c849cbb77199) Prefer tonemapx during HDR image extraction ### 📊 Changes **3 files changed** (+53 additions, -10 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs` (+40 -4) 📝 `MediaBrowser.MediaEncoding/Encoder/EncoderValidator.cs` (+1 -0) 📝 `MediaBrowser.MediaEncoding/Encoder/MediaEncoder.cs` (+12 -6) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> This adds a fast SIMD optimized software tonemap filter support to the server. This filter currently does not support dolby vision reshaping and only support HDR10 and HLG inputs. With current veryfast libx264 settings we can expect 4K60 transcoding+tone mapping on a modern processor with 8 cores equivalent to Zen3/Golden Cove/Firestorm level performance. **Changes** <!-- Describe your changes here in 1-5 sentences. --> - Add tonemapx filter support to software transcode pipeline - Add tonemapx filter support to thumbnail generation pipeline **Issues** <!-- Tag any issues that this PR solves here. ex. Fixes # --> Depends on ffmpeg pr https://github.com/jellyfin/jellyfin-ffmpeg/pull/407 The perf figures in the original post only reflects the initial implementation. The finalized perf figures are here: https://github.com/jellyfin/jellyfin-ffmpeg/pull/407#issuecomment-2198303178 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 07:07:33 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#13123