[FFmpeg-devel] fate/webp : add test for webp lossless rgb decoding

Michael Niedermayer michael at niedermayer.cc
Sun Jun 26 21:13:41 CEST 2016


On Sat, Jun 25, 2016 at 09:56:15PM +0200, Martin Vignali wrote:
> 2016-06-25 21:52 GMT+02:00 Martin Vignali <martin.vignali at gmail.com>:
> 
> >
> >
> > 2016-06-25 21:47 GMT+02:00 James Almer <jamrial at gmail.com>:
> >
> >> On 6/25/2016 4:43 PM, Martin Vignali wrote:
> >> > Hello,
> >> >
> >> > In attach a patch to add a fate test for WebP lossless rgb
> >> >
> >> > Sample can be found here : https://we.tl/dmPsitbwXK
> >> >
> >> > and need to be put inside ./fate-suite/webp (the webp folder doesn't
> >> exists)
> >> >
> >> > to test :
> >> > make fate-webp SAMPLES=fate-suite/
> >> >
> >> >
> >> > I doesn't make fate test, for lossy rgb/rgba, because the actual default
> >> > output of ffmpeg
> >> > have a lower image quality than the official decoder (google dwebp) on
> >> my
> >> > samples
> >> >
> >> > for lossless rgba, i doesn't make fate test, because the alpha value
> >> seems
> >> > to not be the right value
> >>
> >> This is a bug and should be addressed, then. Lossless should mean
> >> lossless.
> >> Can you fix it, or open a ticket on trac with a testcase?
> >>
> >>
> > Sorry, in fact rgba lossless is ok (my mistake)
> >
> > i will open a ticket on trac for lossy
> >
> > and i will make another patch for fate for rgb and rgba lossless webP
> >
> > Martin
> >
> >
> 
> New patch in attach, with rgb, and rgba lossless tests :
> samples can be found here : https://we.tl/98mxCyIkhv
> 
> and need to be put inside ./fate-suite/webp (the webp folder doesn't exists)
> 
> to test :
> make fate-webp SAMPLES=fate-suite/
> 
> 
> Martin

>  fate/image.mak              |   10 ++++++++++
>  ref/fate/webp-rgb-lossless  |    6 ++++++
>  ref/fate/webp-rgba-lossless |    6 ++++++
>  3 files changed, 22 insertions(+)
> 2ae459d785fb28fcc94555b021d5d76a26394801  0001-fate-webp-add-test-for-webp-lossless-decoding-rgb-an.patch
> From 9ec694ad16a680650c8618226c516a6d4ba9889c Mon Sep 17 00:00:00 2001
> From: Martin Vignali <martin.vignali at gmail.com>
> Date: Sat, 25 Jun 2016 21:53:23 +0200
> Subject: [PATCH] fate/webp : add test for webp lossless decoding (rgb and
>  rgba)
> 
> output have been compared with official decoding tool output (dwebp)
> ---
>  tests/fate/image.mak              | 10 ++++++++++
>  tests/ref/fate/webp-rgb-lossless  |  6 ++++++
>  tests/ref/fate/webp-rgba-lossless |  6 ++++++
>  3 files changed, 22 insertions(+)
>  create mode 100644 tests/ref/fate/webp-rgb-lossless
>  create mode 100644 tests/ref/fate/webp-rgba-lossless

applied

thankx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Old school: Use the lowest level language in which you can solve the problem
            conveniently.
New school: Use the highest level language in which the latest supercomputer
            can solve the problem without the user falling asleep waiting.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20160626/825095a3/attachment.sig>


More information about the ffmpeg-devel mailing list