From 7f6e6649cd217997bea383bdbf1a351362717bec Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 4 Aug 2022 17:04:53 +0200
Subject: [PATCH 1/2] let ffmpeg handle scales by forcing original aspect ratio

---
 openpype/plugins/publish/extract_review.py | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/openpype/plugins/publish/extract_review.py b/openpype/plugins/publish/extract_review.py
index 533a87acb4..fe5d34b1a1 100644
--- a/openpype/plugins/publish/extract_review.py
+++ b/openpype/plugins/publish/extract_review.py
@@ -1390,9 +1390,11 @@ class ExtractReview(pyblish.api.InstancePlugin):
             self.log.debug("height_half_pad: `{}`".format(height_half_pad))
 
             filters.extend([
-                "scale={}x{}:flags=lanczos".format(
-                    width_scale, height_scale
-                ),
+                (
+                    "scale={}x{}"
+                    ":flags=lanczos"
+                    ":force_original_aspect_ratio=decrease"
+                ).format(output_width, output_height),
                 "pad={}:{}:{}:{}:{}".format(
                     output_width, output_height,
                     width_half_pad, height_half_pad,

From a0fed43787fab4b945ea850235dde2270d0203b9 Mon Sep 17 00:00:00 2001
From: Jakub Trllo <jakub.trllo@gmail.com>
Date: Thu, 4 Aug 2022 17:07:54 +0200
Subject: [PATCH 2/2] don't even calculate the padded part

---
 openpype/plugins/publish/extract_review.py | 23 +---------------------
 1 file changed, 1 insertion(+), 22 deletions(-)

diff --git a/openpype/plugins/publish/extract_review.py b/openpype/plugins/publish/extract_review.py
index fe5d34b1a1..7442d3aacb 100644
--- a/openpype/plugins/publish/extract_review.py
+++ b/openpype/plugins/publish/extract_review.py
@@ -1369,35 +1369,14 @@ class ExtractReview(pyblish.api.InstancePlugin):
             or input_width != output_width
             or pixel_aspect != 1
         ):
-            if input_res_ratio < output_res_ratio:
-                self.log.debug(
-                    "Input's resolution ratio is lower then output's"
-                )
-                width_scale = int(input_width * scale_factor_by_height)
-                width_half_pad = int((output_width - width_scale) / 2)
-                height_scale = output_height
-                height_half_pad = 0
-            else:
-                self.log.debug("Input is heigher then output")
-                width_scale = output_width
-                width_half_pad = 0
-                height_scale = int(input_height * scale_factor_by_width)
-                height_half_pad = int((output_height - height_scale) / 2)
-
-            self.log.debug("width_scale: `{}`".format(width_scale))
-            self.log.debug("width_half_pad: `{}`".format(width_half_pad))
-            self.log.debug("height_scale: `{}`".format(height_scale))
-            self.log.debug("height_half_pad: `{}`".format(height_half_pad))
-
             filters.extend([
                 (
                     "scale={}x{}"
                     ":flags=lanczos"
                     ":force_original_aspect_ratio=decrease"
                 ).format(output_width, output_height),
-                "pad={}:{}:{}:{}:{}".format(
+                "pad={}:{}:(ow-iw)/2:(oh-ih)/2:{}".format(
                     output_width, output_height,
-                    width_half_pad, height_half_pad,
                     overscan_color_value
                 ),
                 "setsar=1"