[FFmpeg-devel] [PATCH] avcodec/nvenc: Fixing default number of B frames

[Philip Langdale]

On Fri, 19 Feb 2021 18:01:42 +0100
Timo Rothenpieler <timo at rothenpieler.org> wrote:

> On 19.02.2021 16:29, Ricardo Monteiro wrote:
> > Hello,
> > 
> > The patch fixes a bug regarding the number of B frames being used:
> > The default number of B frames is now -1 for both h264_nvenc and
> > hevc_nvenc. This allows the number of B frames to be determined
> > based on the coding preset or overwritten using “-bf” in CLI.  
> 
> The condition change seems unnecessary, given that the new one does
> the exact same logic.
> 
> I'm in favor of changing the bf default, even though it technically 
> changes behavior of existing command lines.
> It seems counter intuitive that the encoder would gimp itself by 
> disabling bframes by default.
> 
> If no one objects, I will apply this.
> 

I can't find the patch. Was it discussed off list? The concept sounds
fine.

--phil