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

[Nicolas George]

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.

> More complex checks can be done by using a combination of lut

Indeed.

> +/**
> + * Calculate the smallest rectangle that will encompass the
> + * region with values != null_val.
> +
> + * @param bbox bounding box structure which is updated with the found values.
> + *             If no pixels could be found with value != null_val, the

The stars seem a little strange.

Apart from that, I do not think I have further remarks. Thanks for your
work.

Regards,

-- 
  Nicolas George
-------------- 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/20120313/525e4d74/attachment.asc>