From 62fb374349e0b13707e12ada5a3410a395c58efa Mon Sep 17 00:00:00 2001 From: "Avi Halachmi (:avih)" Date: Thu, 26 Nov 2020 23:42:20 +0200 Subject: vo_sixel: change default dither to "auto" For two reasons: 1. It was counter intuitive that there's an "auto" value (which is actually a libsixel value and not an mpv one), but it's not the default value - our default was Atkinson. 2. "auto" provides better dithering than Atkinson with libsixel, which is especially noticeable with smooth gradients - where Atkinson has visible banding. In libsixel 1.8.2 the "auto" value maps to Atkinson if the output palette has up to 16 colors, or to Floyd-Steinberg otherwise (e.g. using fixed palette with 256 colors chooses Floyd-Steinberg). --- DOCS/man/vo.rst | 6 +++--- video/out/vo_sixel.c | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/DOCS/man/vo.rst b/DOCS/man/vo.rst index 2802ce914c..265787e73a 100644 --- a/DOCS/man/vo.rst +++ b/DOCS/man/vo.rst @@ -423,12 +423,12 @@ Available video output drivers are: Selects the dither algorithm which libsixel should apply. Can be one of the below list as per libsixel's documentation. - auto - Choose diffuse type automatically + auto (Default) + Let libsixel choose the dithering method. none Don't diffuse atkinson - Diffuse with Bill Atkinson's method. (Default) + Diffuse with Bill Atkinson's method. fs Diffuse with Floyd-Steinberg method jajuni diff --git a/video/out/vo_sixel.c b/video/out/vo_sixel.c index dff20f716d..df0c05e139 100644 --- a/video/out/vo_sixel.c +++ b/video/out/vo_sixel.c @@ -470,7 +470,7 @@ const struct vo_driver video_out_sixel = { .uninit = uninit, .priv_size = sizeof(struct priv), .priv_defaults = &(const struct priv) { - .opt_diffuse = DIFFUSE_ATKINSON, + .opt_diffuse = DIFFUSE_AUTO, .opt_width = 0, .opt_height = 0, .opt_reqcolors = 256, -- cgit v1.2.3