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

[Stefano Sabatini]

On date Tuesday 2012-03-13 19:22:25 +0100, Stefano Sabatini encoded:
> 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.

Applied.
-- 
FFmpeg = Free and Formidable Mere Powerful Evil Gospel