[FFmpeg-devel] [PATCH v2] Allow 'concat' filter to support inputs with different frame rates
Michael Niedermayer
michael at niedermayer.cc
Sat Mar 11 15:28:29 EET 2017
On Thu, Mar 09, 2017 at 03:27:16PM -0500, Calvin Walton wrote:
> Right now, the concat filter does not set the frame_rate value on any of
> the out links. As a result, the default ffmpeg behaviour kicks in - to
> copy the framerate from the first input to the outputs.
>
> If a later input is higher framerate, this results in dropped frames; if
> a later input is lower framerate it might cause judder.
>
> This patch checks if all of the video inputs have the same framerate, and
> if not it sets the out link to use '1/0' as the frame rate, the value
> meaning "unknown/vfr".
>
> A test is added to verify the VFR behaviour. The existing test for CFR
> behaviour passes unchanged.
> ---
> libavfilter/avf_concat.c | 15 ++-
> tests/fate/filter-video.mak | 4 +-
> tests/ref/fate/filter-concat-vfr | 224 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 241 insertions(+), 2 deletions(-)
> create mode 100644 tests/ref/fate/filter-concat-vfr
seem to fail on x86-32
make: *** No rule to make target `tests/data/filtergraphs/concat-vfr', needed by `fate-filter-concat-vfr'. Stop.
make: *** Waiting for unfinished jobs....
[...]
--
Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB
The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170311/e456aae7/attachment.sig>
More information about the ffmpeg-devel
mailing list