[FFmpeg-devel] [PATCH 2/2] fate: add owdenoise test.

[Michael Niedermayer]

On Thu, May 16, 2013 at 09:04:02PM +0200, Clément Bœsch wrote:
> On Thu, May 16, 2013 at 08:27:02PM +0200, Michael Niedermayer wrote:
> > On Thu, May 16, 2013 at 12:58:23AM +0200, Clément Bœsch wrote:
> > > ---
> > >  tests/fate/filter-video.mak            |  3 ++
> > >  tests/filtergraphs/owdenoise           |  1 +
> > >  tests/ref/fate/filter-owdenoise-sample | 74 ++++++++++++++++++++++++++++++++++
> > >  3 files changed, 78 insertions(+)
> > >  create mode 100644 tests/filtergraphs/owdenoise
> > >  create mode 100644 tests/ref/fate/filter-owdenoise-sample
> > > 
> > > diff --git a/tests/fate/filter-video.mak b/tests/fate/filter-video.mak
> > > index 2cf41ff..049ca49 100644
> > > --- a/tests/fate/filter-video.mak
> > > +++ b/tests/fate/filter-video.mak
> > > @@ -96,6 +96,9 @@ fate-filter-unsharp: CMD = framecrc -c:v pgmyuv -i $(SRC) -vf unsharp=11:11:-1.5
> > >  FATE_FILTER-$(call ALLYES, SMJPEG_DEMUXER MJPEG_DECODER PERMS_FILTER HQDN3D_FILTER) += fate-filter-hqdn3d-sample
> > >  fate-filter-hqdn3d-sample: CMD = framecrc -idct simple -i $(SAMPLES)/smjpeg/scenwin.mjpg -filter_complex_script $(SRC_PATH)/tests/filtergraphs/hqdn3d -an
> > >  
> > > +FATE_FILTER-$(call ALLYES, SMJPEG_DEMUXER MJPEG_DECODER PERMS_FILTER OWDENOISE_FILTER) += fate-filter-owdenoise-sample
> > > +fate-filter-owdenoise-sample: CMD = framecrc -idct simple -i $(SAMPLES)/smjpeg/scenwin.mjpg -filter_complex_script $(SRC_PATH)/tests/filtergraphs/owdenoise -an
> > > +
> > >  FATE_FILTER-$(call ALLYES, UTVIDEO_DECODER AVI_DEMUXER PERMS_FILTER CURVES_FILTER) += fate-filter-curves
> > >  fate-filter-curves: CMD = framecrc -i $(SAMPLES)/utvideo/utvideo_rgb_median.avi -vf perms=random,curves=vintage
> > >  
> > > diff --git a/tests/filtergraphs/owdenoise b/tests/filtergraphs/owdenoise
> > > new file mode 100644
> > > index 0000000..9f62e16
> > > --- /dev/null
> > > +++ b/tests/filtergraphs/owdenoise
> > > @@ -0,0 +1 @@
> > > +perms=random, owdenoise=10:20:20:enable='not(between(t,2,3))'
> > > diff --git a/tests/ref/fate/filter-owdenoise-sample b/tests/ref/fate/filter-owdenoise-sample
> > > new file mode 100644
> > > index 0000000..b724f70
> > > --- /dev/null
> > > +++ b/tests/ref/fate/filter-owdenoise-sample
> > > @@ -0,0 +1,74 @@
> > > +#tb 0: 1/9
> > > +0,          0,          0,        1,   115200, 0x2c810465
> > > +0,          1,          1,        1,   115200, 0x2c810465
> > > +0,          2,          2,        1,   115200, 0x87ea4c2f
> > 
> > I get this:
> > 
> > +0,          0,          0,        1,   115200, 0x375ec573
> > +0,          1,          1,        1,   115200, 0x375ec573
> > +0,          2,          2,        1,   115200, 0x9ed6903b
> > +0,          3,          3,        1,   115200, 0xdb29e76d
> > 
> 
> Even with the previous patch adding yuvj? (it is required for scenwin.mjpg
> to avoid the convert).

with the patch it works on x86-64 linux but still fails on x86-32
@@ -1,17 +1,17 @@
 #tb 0: 1/9
 0,          0,          0,        1,   115200, 0x2c810465
 0,          1,          1,        1,   115200, 0x2c810465
-0,          2,          2,        1,   115200, 0x87ea4c2f
-0,          3,          3,        1,   115200, 0x8a254b9d
+0,          2,          2,        1,   115200, 0xc02d4c30
+0,          3,          3,        1,   115200, 0xce0c4b9e
 0,          4,          4,        1,   115200, 0x2926f7bf
 0,          5,          5,        1,   115200, 0xfb1033e1

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130516/72d3037d/attachment.asc>