-----Original Message----- From: ffmpeg-devel [mailto:ffmpeg-devel-boun...@ffmpeg.org] On Behalf Of Mark Thompson Sent: Thursday, February 21, 2019 5:35 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH v2 1/1] avcodec/vaapi_encode: add frame-skip func
On 20/02/2019 10:33, Jing SUN wrote: > This implements app controlled frame skipping in vaapi encoding. To > make a frame skipped, allocate its frame side data of the newly added > AV_FRAME_DATA_SKIP_FRAME type and set its value to 1. > > Signed-off-by: Jing SUN <jing.a....@intel.com> > --- > libavcodec/vaapi_encode.c | 112 > ++++++++++++++++++++++++++++++++++++++++++++-- > libavcodec/vaapi_encode.h | 5 +++ > libavutil/frame.c | 1 + > libavutil/frame.h | 5 +++ > 4 files changed, 119 insertions(+), 4 deletions(-) Have a look at <https://lists.ffmpeg.org/pipermail/ffmpeg-devel/2019-February/239989.html>, which tries to implement this feature in a more general way without adding any ad-hoc API. - Mark -----Original Message----- Hi Mark, I think the 239989 patch is quite a good one to implement VFR by adapting VAAPI frame-skip feature, but it does not fulfill the feature requirement that we have been asked to achieve, which is to skip the frames that are selected by apps, and the purpose to skip any frames is not just to control frame-rate, but also to skip duplicated ones, or to skip abandoned ones due to some changes just made, etc. Could we please keep both and let users decide which way they want to use? Regards, SUN, Jing _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
