[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