[Ffmpeg-devel] [PATCH] tunable vb_strategy=1

[Matthias Wieser]

Corey Hickey wrote:
> Corey Hickey wrote:
> > It seems to me that vb_strategy=1 is fundamentally a good idea, but
> > it's too sensitive. I looked in the code and found a number: 40. If
> > that number were lower, lavc would use more B-frames. I made it tunable
> > and ran some tests. All results are for the second pass (except
> > first-pass encoding time), and each test had vmax_b_frames=2.
>
> Would anybody object if I applied this? If nobody speaks up, then I'll
> apply it in one day.

I'm not a developer but I have done some tests and lower numbers at least do 
not look worse and give better PSNR (hello Rich).

So it would be nice if you would not only apply the patch but also adjust 
the default to some lower number, e.g. 10. It's all about sane defaults.

Regards,
  Matthias