[FFmpeg-devel] [PATCH] doc/encoders: correct the description for ts_target_bitrate

[James Zern]

Hi,

On Wed, Dec 18, 2019 at 2:00 PM Wonkap Jang
<wonkap-at-google.com at ffmpeg.org> wrote:
>
> Hi James,
>
> I have updated the libvpx.
> https://chromium-review.googlesource.com/c/webm/libvpx/+/1974899
>

Thanks for posting the patch. Remember top-posting isn't preferred on
this mailing list [1].

[1] http://ffmpeg.org/contact.html#MailingLists

> Thank you,
>
> Wonkap
>
> On Wed, Dec 18, 2019 at 1:39 PM James Zern <jzern-at-google.com at ffmpeg.org>
> wrote:
>
> > Hi,
> >
> > On Wed, Dec 18, 2019 at 1:17 PM Wonkap Jang
> > <wonkap-at-google.com at ffmpeg.org> wrote:
> > >
> > > ts_target_bitrate is in kbps, not bps. This commit clarifies the unit
> > > and modifies the example to match the description.
> > > ---
> > >  doc/encoders.texi | 5 +++--
> > >  1 file changed, 3 insertions(+), 2 deletions(-)
> > >
> >
> > lgtm if you can verify that with the libvpx implementation and update
> > the docs [1].
> >
> > [1]
> > https://chromium.googlesource.com/webm/libvpx/+/refs/heads/master/vpx/vpx_encoder.h#646
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".