summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Ram Mohan <ram.mohan@ittiam.com> 2023-06-11 07:07:27 +0530
committer Dichen Zhang <dichenzhang@google.com> 2023-06-13 17:25:35 +0000
commit1790bb5d5bd420047ed14ec0bfd2c73478c747d0 (patch)
treecb8fb991ce5ec73e847e15348d731759cbf3894f
parent69d1bc228a1fb72ff8539766adb0b771b0c1c935 (diff)
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
-rw-r--r--libs/ultrahdr/jpegdecoderhelper.cpp2
-rw-r--r--libs/ultrahdr/jpegencoderhelper.cpp2
2 files changed, 2 insertions, 2 deletions
diff --git a/libs/ultrahdr/jpegdecoderhelper.cpp b/libs/ultrahdr/jpegdecoderhelper.cpp
index fac90c503d..6c3a8c10b7 100644
--- a/libs/ultrahdr/jpegdecoderhelper.cpp
+++ b/libs/ultrahdr/jpegdecoderhelper.cpp
@@ -242,7 +242,7 @@ bool JpegDecoderHelper::decode(const void* image, int length, bool decodeToRGBA)
cinfo.raw_data_out = TRUE;
}
- cinfo.dct_method = JDCT_IFAST;
+ cinfo.dct_method = JDCT_ISLOW;
jpeg_start_decompress(&cinfo);
diff --git a/libs/ultrahdr/jpegencoderhelper.cpp b/libs/ultrahdr/jpegencoderhelper.cpp
index 10a763035f..867ffbb4e9 100644
--- a/libs/ultrahdr/jpegencoderhelper.cpp
+++ b/libs/ultrahdr/jpegencoderhelper.cpp
@@ -139,7 +139,7 @@ void JpegEncoderHelper::setJpegCompressStruct(int width, int height, int quality
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