[FFmpeg-devel] [PATCH] doc/encoders: add VP9 temporal scalability encoding option

[James Zern]

Hi,

On Mon, Dec 16, 2019 at 2:21 PM Wonkap Jang
<wonkap-at-google.com at ffmpeg.org> wrote:
>
> Documentation change for adding support for encoding
> with temporal scalability in VP9.
> ---

This part sounds related to the other patches you're sending. The
documentation can be added with the code change for vp9.

>  doc/encoders.texi | 23 ++++++++++++++++++-----
>  1 file changed, 18 insertions(+), 5 deletions(-)
>
> diff --git a/doc/encoders.texi b/doc/encoders.texi
> index 4ee518a124..7efc446a6c 100644
> --- a/doc/encoders.texi
> +++ b/doc/encoders.texi
> [...]
>  ffmpeg -i INPUT -c:v libvpx -ts-parameters ts_number_layers=3:\
> -ts_target_bitrate=250000,500000,1000000:ts_rate_decimator=4,2,1:\
> -ts_periodicity=4:ts_layer_id=0,2,1,2 OUTPUT
> +ts_target_bitrate=250,500,1000:ts_rate_decimator=4,2,1:\
> +ts_periodicity=4:ts_layer_id=0,2,1,2:ts_layering_mode=3 OUTPUT
>  @end example
>  Below is a brief explanation of each of the parameters, please
>  refer to @code{struct vpx_codec_enc_cfg} in @code{vpx/vpx_encoder.h} for more
> @@ -1903,13 +1901,28 @@ details.
>  @item ts_number_layers
>  Number of temporal coding layers.
>  @item ts_target_bitrate
> -Target bitrate for each temporal layer.
> +Target bitrate for each temporal layer in kbps.
> +(bitrate should be inclusive of the lower temporal layer).

This portion is a bug fix for the existing docs (bps -> kbps). Can you
split this out to its own patch?