Re: [FFmpeg-devel] [PATCH] Change libaom default to crf=32.

2019-08-23 Thread James Zern
On Wed, Aug 21, 2019 at 6:03 PM James Zern wrote: > > On Wed, Aug 21, 2019 at 12:18 PM Elliott Karpilovsky > wrote: > > > > From: elliottk > > > > Current default is 256kbps, which produces inconsistent > > results (too high for low-res, too low for hi-res). > > Use CRF instead, which will adapt

Re: [FFmpeg-devel] [PATCH] Change libaom default to crf=32.

2019-08-22 Thread James Zern
Hi, On Thu, Aug 22, 2019 at 12:25 AM Moritz Barsnick wrote: > > On Wed, Aug 21, 2019 at 18:03:23 -0700, James Zern wrote: > > > From: elliottk > > > Current default is 256kbps, which produces inconsistent > > > results (too high for low-res, too low for hi-res). > > > Use CRF instead, which will

Re: [FFmpeg-devel] [PATCH] Change libaom default to crf=32.

2019-08-22 Thread Moritz Barsnick
On Wed, Aug 21, 2019 at 18:03:23 -0700, James Zern wrote: > > From: elliottk > > Current default is 256kbps, which produces inconsistent > > results (too high for low-res, too low for hi-res). > > Use CRF instead, which will adapt. > > lgtm. I'll apply this soon if there aren't any other comments.

Re: [FFmpeg-devel] [PATCH] Change libaom default to crf=32.

2019-08-21 Thread James Zern
On Wed, Aug 21, 2019 at 12:18 PM Elliott Karpilovsky wrote: > > From: elliottk > > Current default is 256kbps, which produces inconsistent > results (too high for low-res, too low for hi-res). > Use CRF instead, which will adapt. > --- > libavcodec/libaomenc.c | 9 + > 1 file changed, 5

[FFmpeg-devel] [PATCH] Change libaom default to crf=32.

2019-08-21 Thread Elliott Karpilovsky
From: elliottk Current default is 256kbps, which produces inconsistent results (too high for low-res, too low for hi-res). Use CRF instead, which will adapt. --- libavcodec/libaomenc.c | 9 + 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/libavcodec/libaomenc.c b/libavcode