Add stereo downmix algorithm selection.

Jellyfin.Api/Controllers/DynamicHlsController.cs

@@ -22,6 +22,7 @@ using MediaBrowser.Controller.Library;
 using MediaBrowser.Controller.MediaEncoding;
 using MediaBrowser.Model.Configuration;
 using MediaBrowser.Model.Dlna;
+using MediaBrowser.Model.Entities;
 using MediaBrowser.Model.IO;
 using MediaBrowser.Model.Net;
 using Microsoft.AspNetCore.Authorization;
@@ -1731,7 +1732,12 @@ namespace Jellyfin.Api.Controllers
 
             var channels = state.OutputAudioChannels;
 
-            if (channels.HasValue)
+            if (channels.HasValue
+                && (channels.Value != 2
+                    || (state.AudioStream is not null
+                        && state.AudioStream.Channels.HasValue
+                        && state.AudioStream.Channels.Value > 5
+                        && _encodingOptions.DownMixStereoAlgorithm == DownMixStereoAlgorithms.None)))
             {
                 args += " -ac " + channels.Value;
             }

MediaBrowser.Controller/MediaEncoding/EncodingHelper.cs

@@ -2129,17 +2129,32 @@ namespace MediaBrowser.Controller.MediaEncoding
 
             var filters = new List<string>();
 
-            // Boost volume to 200% when downsampling from 6ch to 2ch
             if (channels.HasValue
-                && channels.Value <= 2
+                && channels.Value == 2
                 && state.AudioStream is not null
                 && state.AudioStream.Channels.HasValue
-                && state.AudioStream.Channels.Value > 5
+                && state.AudioStream.Channels.Value > 5)
-                && !encodingOptions.DownMixAudioBoost.Equals(1))
+            {
+                switch (encodingOptions.DownMixStereoAlgorithm)
+                {
+                    case DownMixStereoAlgorithms.Dave750:
+                        filters.Add("volume=4.25");
+                        filters.Add("pan=stereo|c0=0.5*c2+0.707*c0+0.707*c4+0.5*c3|c1=0.5*c2+0.707*c1+0.707*c5+0.5*c3");
+                        break;
+                    case DownMixStereoAlgorithms.NightmodeDialogue:
+                        filters.Add("pan=stereo|c0=c2+0.30*c0+0.30*c4|c1=c2+0.30*c1+0.30*c5");
+                        break;
+                    case DownMixStereoAlgorithms.None:
+                    default:
+                        if (!encodingOptions.DownMixAudioBoost.Equals(1))
                         {
                             filters.Add("volume=" + encodingOptions.DownMixAudioBoost.ToString(CultureInfo.InvariantCulture));
                         }
 
+                        break;
+                }
+            }
+
             var isCopyingTimestamps = state.CopyTimestamps || state.TranscodingType != TranscodingJobType.Progressive;
             if (state.SubtitleStream is not null && state.SubtitleStream.IsTextSubtitleStream && state.SubtitleDeliveryMethod == SubtitleDeliveryMethod.Encode && !isCopyingTimestamps)
             {
@@ -5711,10 +5726,9 @@ namespace MediaBrowser.Controller.MediaEncoding
                 return args;
             }
 
-            // Add the number of audio channels
             var channels = state.OutputAudioChannels;
 
-            if (channels.HasValue)
+            if (channels.HasValue && ((channels.Value != 2 && state.AudioStream.Channels <= 5) || encodingOptions.DownMixStereoAlgorithm == DownMixStereoAlgorithms.None))
             {
                 args += " -ac " + channels.Value;
             }

MediaBrowser.Model/Configuration/EncodingOptions.cs

@@ -1,14 +1,21 @@
 #nullable disable
-#pragma warning disable CS1591
+using MediaBrowser.Model.Entities;
 
-namespace MediaBrowser.Model.Configuration
+namespace MediaBrowser.Model.Configuration;
-{
+
+/// <summary>
+/// Class EncodingOptions.
+/// </summary>
 public class EncodingOptions
 {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="EncodingOptions" /> class.
+    /// </summary>
     public EncodingOptions()
     {
         EnableFallbackFont = false;
         DownMixAudioBoost = 2;
+        DownMixStereoAlgorithm = DownMixStereoAlgorithms.None;
         MaxMuxingQueueSize = 2048;
         EnableThrottling = false;
         ThrottleDelaySeconds = 180;
@@ -43,22 +50,54 @@ namespace MediaBrowser.Model.Configuration
         HardwareDecodingCodecs = new string[] { "h264", "vc1" };
     }
 
+    /// <summary>
+    /// Gets or sets the thread count used for encoding.
+    /// </summary>
     public int EncodingThreadCount { get; set; }
 
+    /// <summary>
+    /// Gets or sets the temporary transcoding path.
+    /// </summary>
     public string TranscodingTempPath { get; set; }
 
+    /// <summary>
+    /// Gets or sets the path to the fallback font.
+    /// </summary>
     public string FallbackFontPath { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether to use the fallback font.
+    /// </summary>
     public bool EnableFallbackFont { get; set; }
 
+    /// <summary>
+    /// Gets or sets the audio boost applied when downmixing audio.
+    /// </summary>
     public double DownMixAudioBoost { get; set; }
 
+    /// <summary>
+    /// Gets or sets the algorithm used for downmixing audio to stereo.
+    /// </summary>
+    public DownMixStereoAlgorithms DownMixStereoAlgorithm { get; set; }
+
+    /// <summary>
+    /// Gets or sets the maximum size of the muxing queue.
+    /// </summary>
     public int MaxMuxingQueueSize { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether throttling is enabled.
+    /// </summary>
     public bool EnableThrottling { get; set; }
 
+    /// <summary>
+    /// Gets or sets the delay after which throttling happens.
+    /// </summary>
     public int ThrottleDelaySeconds { get; set; }
 
+    /// <summary>
+    /// Gets or sets the hardware acceleration type.
+    /// </summary>
     public string HardwareAccelerationType { get; set; }
 
     /// <summary>
@@ -71,58 +110,138 @@ namespace MediaBrowser.Model.Configuration
     /// </summary>
     public string EncoderAppPathDisplay { get; set; }
 
+    /// <summary>
+    /// Gets or sets the VA-API device.
+    /// </summary>
     public string VaapiDevice { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether tonemapping is enabled.
+    /// </summary>
     public bool EnableTonemapping { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether VPP tonemapping is enabled.
+    /// </summary>
     public bool EnableVppTonemapping { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping algorithm.
+    /// </summary>
     public string TonemappingAlgorithm { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping range.
+    /// </summary>
     public string TonemappingRange { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping desaturation.
+    /// </summary>
     public double TonemappingDesat { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping threshold.
+    /// </summary>
     public double TonemappingThreshold { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping peak.
+    /// </summary>
     public double TonemappingPeak { get; set; }
 
+    /// <summary>
+    /// Gets or sets the tone-mapping parameters.
+    /// </summary>
     public double TonemappingParam { get; set; }
 
+    /// <summary>
+    /// Gets or sets the VPP tone-mapping brightness.
+    /// </summary>
     public double VppTonemappingBrightness { get; set; }
 
+    /// <summary>
+    /// Gets or sets the VPP tone-mapping contrast.
+    /// </summary>
     public double VppTonemappingContrast { get; set; }
 
+    /// <summary>
+    /// Gets or sets the H264 CRF.
+    /// </summary>
     public int H264Crf { get; set; }
 
+    /// <summary>
+    /// Gets or sets the H265 CRF.
+    /// </summary>
     public int H265Crf { get; set; }
 
+    /// <summary>
+    /// Gets or sets the encoder preset.
+    /// </summary>
     public string EncoderPreset { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether the framerate is doubled when deinterlacing.
+    /// </summary>
     public bool DeinterlaceDoubleRate { get; set; }
 
+    /// <summary>
+    /// Gets or sets the deinterlace method.
+    /// </summary>
     public string DeinterlaceMethod { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether 10bit HEVC decoding is enabled.
+    /// </summary>
     public bool EnableDecodingColorDepth10Hevc { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether 10bit VP9 decoding is enabled.
+    /// </summary>
     public bool EnableDecodingColorDepth10Vp9 { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether the enhanced NVDEC is enabled.
+    /// </summary>
     public bool EnableEnhancedNvdecDecoder { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether the system native hardware decoder should be used.
+    /// </summary>
     public bool PreferSystemNativeHwDecoder { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether the Intel H264 low-power hardware encoder should be used.
+    /// </summary>
     public bool EnableIntelLowPowerH264HwEncoder { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether the Intel HEVC low-power hardware encoder should be used.
+    /// </summary>
     public bool EnableIntelLowPowerHevcHwEncoder { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether hardware encoding is enabled.
+    /// </summary>
     public bool EnableHardwareEncoding { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether HEVC encoding is enabled.
+    /// </summary>
     public bool AllowHevcEncoding { get; set; }
 
+    /// <summary>
+    /// Gets or sets a value indicating whether subtitle extraction is enabled.
+    /// </summary>
     public bool EnableSubtitleExtraction { get; set; }
 
+    /// <summary>
+    /// Gets or sets the codecs hardware encoding is used for.
+    /// </summary>
     public string[] HardwareDecodingCodecs { get; set; }
 
+    /// <summary>
+    /// Gets or sets the file extensions on-demand metadata based keyframe extraction is enabled for.
+    /// </summary>
     public string[] AllowOnDemandMetadataBasedKeyframeExtractionForExtensions { get; set; }
 }
-}

MediaBrowser.Model/Entities/DownMixStereoAlgorithms.cs

@@ -0,0 +1,23 @@
+namespace MediaBrowser.Model.Entities;
+
+/// <summary>
+/// An enum representing an algorithm to downmix 6ch+ to stereo.
+/// Algorithms sourced from https://superuser.com/questions/852400/properly-downmix-5-1-to-stereo-using-ffmpeg/1410620#1410620.
+/// </summary>
+public enum DownMixStereoAlgorithms
+{
+    /// <summary>
+    /// No special algorithm.
+    /// </summary>
+    None = 0,
+
+    /// <summary>
+    /// Algorithm by Dave_750.
+    /// </summary>
+    Dave750 = 1,
+
+    /// <summary>
+    /// Nightmode Dialogue algorithm.
+    /// </summary>
+    NightmodeDialogue = 2
+}