[FFmpeg-devel] [PATCH] lavf/mov.c: Use the correct timescale when seeking for audio.

Sasi Inguva isasi at google.com
Tue Nov 1 19:01:24 EET 2016


Yes.  This is expected. The audio has an edit list with duration 7294805
which at a timescale of 600 means 1324.675 .
Entry[0].SegmentDuration 794805
Entry[0].MediaTime 2048


~/ffmpeg_new/ffprobe -show_frames -print_format compact -select_streams 1
 ~/Downloads/edit_list_sample.mp4

When we add up  (last pts + 0.023 )  it amounts to that . We fix the last
packet duration according to edit list segment duration in our edit list
patch.  Here last pts = 1324.652  .

diff <(~/ffmpeg_old/ffprobe -show_frames -print_format compact
-select_streams 1  ~/Downloads/edit_list_sample.mp4   )
<(~/ffmpeg_new/ffprobe -show_frames -print_format compact -select_streams 1
 ~/Downloads/edit_list_sample.mp4   )

< frame|media_type=audio|stream_index=1|key_frame=1|pkt_pts=
58417152|pkt_pts_time=1324.651973|pkt_dts=58417152|pkt_
dts_time=1324.651973|best_effort_timestamp=58417152|
best_effort_timestamp_time=1324.651973|pkt_duration=3072|
pkt_duration_time=0.069660|pkt_pos=217515035|pkt_size=
424|sample_fmt=fltp|nb_samples=1024|channels=2|channel_layout=stereo
---
> frame|media_type=audio|stream_index=1|key_frame=1|pkt_pts=
58417152|pkt_pts_time=1324.651973|pkt_dts=58417152|pkt_
dts_time=1324.651973|best_effort_timestamp=58417152|
best_effort_timestamp_time=1324.651973|pkt_duration=1016|
pkt_duration_time=0.023039|pkt_pos=217515035|pkt_size=
424|sample_fmt=fltp|nb_samples=1024|channels=2|channel_layout=stereo



On Tue, Nov 1, 2016 at 7:01 AM, Derek Buitenhuis <derek.buitenhuis at gmail.com
> wrote:

> On 10/31/2016 1:48 PM, Rostislav Pehlivanov wrote:
> > The tests pass on my machine, pushed after Derek said it looks correct.
>
> Hi Sasi,
>
> I notice there's still one difference after this patch. Could you
> explain if this is intended:
>
>     -            "duration": 3072,
>     -            "duration_time": "0.069660",
>     +            "duration": 1016,
>     +            "duration_time": "0.023039",
>
> This is on the very last packet of the file (the same file I sent you
> before).
>
> Cheers,
> - Derek
>
>


More information about the ffmpeg-devel mailing list