[x265] [PATCH STABLE] encoder: disable lookahead for all-intra encodes

[Steve Borho]

On Wed, Sep 25, 2013 at 1:47 AM, Jason Garrett-Glaser <jason at x264.com>wrote:

> On Tue, Sep 24, 2013 at 11:28 PM, Steve Borho <steve at borho.org> wrote:
> > # HG changeset patch
> > # User Steve Borho <steve at borho.org>
> > # Date 1380090499 18000
> > #      Wed Sep 25 01:28:19 2013 -0500
> > # Branch stable
> > # Node ID 91ae57f62e18e447b217bbfe9f4d45f36dd19b03
> > # Parent  ddb7abf83f983a5d5e1e0e79b57073d5bd9c8160
> > encoder: disable lookahead for all-intra encodes
> >
> > This works around a bug in the current Lookahead::slicetypeDecide().  I
> have
> > a patch in the works which fixes this properly, but I think this is a
> reasonable
> > workaround (who needs lookahead or lowres structs for all-intra encodes)?
>
> At least in x264, lookahead is used for AQ, VBV, and ratecontrol in
> all-intra encodes.
>

Good points.  Currently our rate control calls back into lookahead for the
frame cost estimate and it can measure the cost then if it hadn't already.
 My patch will remove the need for this as well.  I do need to verify ABR
works correctly with this patch applied.

AQ and VBV are still in the planning stage.

--
Steve
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20130925/7f49c54b/attachment-0001.html>