[FFmpeg-devel] [PATCH] lavfi: add bbox filter

[Stefano Sabatini]

On date Tuesday 2012-03-13 17:32:35 +0100, Michael Niedermayer encoded:
> On Tue, Mar 13, 2012 at 10:19:56AM +0100, Nicolas George wrote:
> > Le duodi 22 ventôse, an CCXX, Stefano Sabatini a écrit :
> > > Uhm... extend the ffprobe writers API?
> > 
> > Actually, I do not know.
> > 
> > > Indeed. Now I could drop support to chroma bbox detection or I could
> > > extend the calculate_bounding_box() function to keep two values
> > > (min_val and max_val), or check if the value is != null_value (I
> > > adopted this variant).
> > 
> > I am not sure, but I believe testing only luminance is the best course for
> > now: after all, very dark is very dark, even if it is very dark saturated
> > red.
> 
> I think the best would be to choose a good model for black and one
> for actual content and then choose the box so as to maximize some
> mathcing score of the models. But this is something for after
> commiting the filter IMHO

Patch updated, I removed support to chroma plane testing, I'll apply
in a few days if I read no comments.
-- 
FFmpeg = Furious Foolish Marvellous Picky Erotic Guru
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-lavfi-add-bbox-filter.patch
Type: text/x-diff
Size: 10391 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120313/d93250a8/attachment.bin>