[FFmpeg-devel] [PATCH] fate: Test pullup and fieldmatch with 5 instead of 1 frame

Michael Niedermayer michaelni at gmx.at
Fri May 15 03:35:24 CEST 2015


On Thu, May 14, 2015 at 01:38:00PM +0200, Clément Bœsch wrote:
> On Tue, May 12, 2015 at 12:29:10PM +0200, Michael Niedermayer wrote:
> > On Tue, May 12, 2015 at 10:21:47AM +0200, Clément Bœsch wrote:
> > > On Mon, May 11, 2015 at 12:59:57PM +0200, Michael Niedermayer wrote:
> > > > On Mon, May 11, 2015 at 07:34:11AM +0200, Clément Bœsch wrote:
> > > > > On Mon, May 11, 2015 at 12:44:14AM +0200, Michael Niedermayer wrote:
> > > > > > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > > > > > ---
> > > > > >  tests/fate-run.sh                        |    3 ++-
> > > > > >  tests/fate/filter-video.mak              |    4 ++--
> > > > > >  tests/ref/fate/filter-pixfmts-fieldmatch |   10 +++++-----
> > > > > >  tests/ref/fate/filter-pixfmts-pullup     |   24 ++++++++++++------------
> > > > > >  4 files changed, 21 insertions(+), 20 deletions(-)
> > > > > > 
> > > > > 
> > > > > Hum, 5 frames test with telecine... I'd really go for more.
> > > > 
> > > > ok, how many frames should we use ?
> > > > 
> > > 
> > > Dunno, maybe 15 to 25.
> > 
> > applied with that change
> > 
> > 
> > > 
> > > BTW, I have the feeling that 5 frames is not really enough for testing a
> > > some of the other filters as well. The first two things that come to my
> > > mind are:
> > > 
> > > - the filters that rely on perms=random might not have enough coverage on
> > >   the permissions
> > > - the filters that use video samples might not come across
> > >   interesting/appropriate frames (whether it's based on the content or the
> > >   properties of frames such as P/B/I for pp filters)
> > > 
> > > Fortunately, pixfmts and video_filter command seems not used that much.
> > 
> > are there any more tests which need changes ?
> 
> Maybe these 3:
> 
> fate/filter-video.mak:fate-filter-edgedetect: CMD = video_filter "format=gray,perms=random,edgedetect"
> fate/filter-video.mak:fate-filter-edgedetect-colormix: CMD = video_filter "format=gbrp,perms=random,edgedetect=mode=colormix"
> fate/filter-video.mak:fate-filter-hue: CMD = video_filter "perms=random,hue=s=sin(2*PI*t)+1"

changed

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150515/851cacd2/attachment.asc>


More information about the ffmpeg-devel mailing list