commit 1790bb5d5bd420047ed14ec0bfd2c73478c747d0
author Ram Mohan <ram.mohan@ittiam.com> Sun Jun 11 07:07:27 2023 +0530
committer Dichen Zhang <dichenzhang@google.com> Tue Jun 13 17:25:35 2023 +0000
tree cb8fb991ce5ec73e847e15348d731759cbf3894f
parent 69d1bc228a1fb72ff8539766adb0b771b0c1c935

ultrahdr: do not select less accurate integer method during dct

At high quality setting, less accurate integer method can have
larger psnr drops. Switch to default setting

Bug: 286617381
Test: ./libultrahdr_app -p inp_p010.yuv -y inp_420p.yuv -w 1920 -h 1080 -o 0

Change-Id: Ide126b87262e1c9a20edca7873b6ca27fc52b2cb

libs/ultrahdr/jpegencoderhelper.cpp

diff --git a/libs/ultrahdr/jpegencoderhelper.cpp b/libs/ultrahdr/jpegencoderhelper.cpp
index 10a7630..867ffbb 100644
--- a/libs/ultrahdr/jpegencoderhelper.cpp
+++ b/libs/ultrahdr/jpegencoderhelper.cpp
@@ -139,7 +139,7 @@
     jpeg_set_quality(cinfo, quality, TRUE);
     jpeg_set_colorspace(cinfo, isSingleChannel ? JCS_GRAYSCALE : JCS_YCbCr);
     cinfo->raw_data_in = TRUE;
-    cinfo->dct_method = JDCT_IFAST;
+    cinfo->dct_method = JDCT_ISLOW;
 
     if (!isSingleChannel) {
         // Configure sampling factors. The sampling factor is JPEG subsampling 420 because the