diff options
author | Graham Booker <gbooker@tamu.edu> | 2014-04-11 23:26:46 -0500 |
---|---|---|
committer | Michael Niedermayer <michaelni@gmx.at> | 2014-04-22 16:05:25 +0200 |
commit | 60fcc19b9068614f25cf64dff5e4aa0e8dbff6a5 (patch) | |
tree | 77315777763128ba1c2ed55bbbadf7229ce79d91 /tests | |
parent | 5809c2a99d9682f64f77b0ad23fa907aa9e8fce2 (diff) | |
download | ffmpeg-streaming-60fcc19b9068614f25cf64dff5e4aa0e8dbff6a5.zip ffmpeg-streaming-60fcc19b9068614f25cf64dff5e4aa0e8dbff6a5.tar.gz |
avformat/mpegtsenc: Changed Video PES packet length to 0.
The rational for this is another issue that plex has exposed. When it is
conducting a transcode of video to HLS for streaming, my father noticed
artifacts when played on his GoogleTV (NSZ-GT1). He sent me a test file
and I reproduced it on my device of the same model. It is important to
note that the artifacts were not present when streaming to VLC or QuickTime
Player. I copied the command-line that plex used, and conducted all of the
following tests using FFmpeg git.
Transcode to HLS: artifacts on playback
Transcode to TS: playback is fine
Cat HLS segments into a single TS: playback is fine
Segment single TS file to segments: artifacts on playback
Segment single TS file to segments using Apple's HLS segmenter: playback is
fine
At this point I carefully examined the differences between Apple's HLS
segmenter output and FFmpeg's. Among the considerable differences, I
noticed that the video PES packets always had a 0 length. So I continued:
Transcode to HLS using FFmpeg with 0 length PES packets: playback is fine.
Segment single TS to segments with 0 length PES packets: playback is fine.
All failures mentioned are only on the GTV since it is the only player on
which I could reproduce artifacts. I only tested the GTV, VLC, and
QuickTime Player though, so my test case is limited. I do not know if
other players exhibit this issue.
Since it was useful last time, I have uploaded the test file as
hls_pes_packet_length.m4v along with its associated txt file which contains
the transcode command-line that was used.
Reviewed-by: Kieran Kunhya <kierank@obe.tv>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/ref/lavf/ts | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/ref/lavf/ts b/tests/ref/lavf/ts index d457dfa..1381f09 100644 --- a/tests/ref/lavf/ts +++ b/tests/ref/lavf/ts @@ -1,3 +1,3 @@ -e0738e00907d371ffbdee5caeab4277d *./tests/data/lavf/lavf.ts +cca6bca512605bbde20b7aa5cccf4850 *./tests/data/lavf/lavf.ts 407020 ./tests/data/lavf/lavf.ts ./tests/data/lavf/lavf.ts CRC=0x71287e25 |