On Thu, Sep 23, 2021 at 4:46 PM Christopher Degawa <[email protected]> wrote: > > On Sun, Sep 19, 2021 at 2:02 PM Christopher Degawa <[email protected]> > wrote: > > > > > > > On Fri, Sep 17, 2021 at 9:28 PM <[email protected]> wrote: > > > >> From: Limin Wang <[email protected]> > >> > >> Signed-off-by: Limin Wang <[email protected]> > >> > > As a note, I personally favor this patch over mine since I don't think > > `enable_tpl_la` is worth exposing to the user if its main role is to switch > > between crf and cqp > > > > Ping on this patch, this has been requested on SVT-AV1's side as well
I think my further comments on the previous version were mostly to move off from the rc option for rate control, which has become more and more seemingly unnecessary (and different from most other encoders for no obvious reason). The only option that with a quick look doesn't match just setting quantizer|bit rate|CRF is "cvbr", but it doesn't seem to be impossible to either rework "rc" for it, or to utilize a separate option for it. Basically the idea is to get it more or less to match other encoders, so that the wrapper doesn't feel completely alien to the rest of things. Jan _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
