From a04adf45c042d4a6749ecd2c285c925a4e72d605 Mon Sep 17 00:00:00 2001 From: Jason Dove <1695733+jasongdove@users.noreply.github.com> Date: Mon, 11 Aug 2025 17:39:01 +0000 Subject: [PATCH] fix green padding with vaapi i965 driver (#2298) --- CHANGELOG.md | 1 + ErsatzTV.FFmpeg/Pipeline/VaapiPipelineBuilder.cs | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 8f1a9caf7..90b673eb2 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -40,6 +40,7 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/). - YAML playouts in particular should build significantly faster - Fix channel playout mode `On Demand` for Block and YAML schedules - Fix QSV transitions when remote streaming from a media server +- Fix green output when padding with VAAPI accel and i965 driver ### Changed - Allow multiple watermarks in playback troubleshooting diff --git a/ErsatzTV.FFmpeg/Pipeline/VaapiPipelineBuilder.cs b/ErsatzTV.FFmpeg/Pipeline/VaapiPipelineBuilder.cs index 525811b8a..3ddf09142 100644 --- a/ErsatzTV.FFmpeg/Pipeline/VaapiPipelineBuilder.cs +++ b/ErsatzTV.FFmpeg/Pipeline/VaapiPipelineBuilder.cs @@ -575,13 +575,13 @@ public class VaapiPipelineBuilder : SoftwarePipelineBuilder if (desiredState.CroppedSize.IsNone && currentState.PaddedSize != desiredState.PaddedSize) { // pad_vaapi seems to pad with green when input is HDR - // also green with 10-bit content and i965 + // also green with i965 driver // so use software pad in these cases - bool is10Bit965 = currentState.BitDepth == 10 && ffmpegState.VaapiDriver + bool is965 = ffmpegState.VaapiDriver .IfNone(string.Empty) .Contains("i965", StringComparison.OrdinalIgnoreCase); - if (isHdrTonemap || is10Bit965) + if (isHdrTonemap || is965) { var padStep = new PadFilter(currentState, desiredState.PaddedSize); currentState = padStep.NextState(currentState);