Opened 9 years ago

Closed 7 years ago

#6833 closed defect (fixed)

setpts filter incorrect handle current_pts value

Reported by: Aleksey Vasenev Owned by:
Priority: important Component: avfilter
Version: git-master Keywords: fps setpts regression
Cc: developpeur@telequid.com Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: no

Description

Summary of the bug:
fps filter use current_pts to duplicate frames at end, but current_pts > last pts after setpts filter and fps insert too many frames at end.
Example must be 5 sec, but 10 actually
How to reproduce:

% ffmpeg -f lavfi -i testsrc=duration=10 -vf trim=5,setpts=PTS-5/TB,fps=25 -pix_fmt yuv420p 1.mkv
ffmpeg version N-88804-g3af2bf0af0

Change History (5)

comment:1 by Cigaes, 9 years ago

Status: new → open
Version: unspecified → git-master

Indeed, that would have that consequence. I will try to find a solution.

comment:2 by Carl Eugen Hoyos, 8 years ago

Keywords: fps setpts regression added
Priority: normal → important
Reproduced by developer: set

Regression since eea64ef4cfb593cbe28465f45e6bd4c41a79cae1

comment:3 by Telequid, 8 years ago

Cc: developpeur@telequid.com added

comment:4 by Telequid, 8 years ago

Note: with latest FFmpeg (4.0.2), calling the fps filter before setpts works (output is 5 seconds long).

ffmpeg -f lavfi -i testsrc=duration=10 -vf trim=5,fps=25,setpts=PTS-5/TB -pix_fmt yuv420p 1.mkv

The command line provided in the OP still doesn't work.

comment:5 by Carl Eugen Hoyos, 7 years ago

Resolution: → fixed
Status: open → closed

Fixed by Paul in 2a546fb7d5722c306dd42c715137e5e493b0d5be

Note: See TracTickets for help on using tickets.