• Graham Booker's avatar
    avformat/mpegtsenc: Changed Video PES packet length to 0. · 60fcc19b
    Graham Booker authored
    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: 's avatarKieran Kunhya <kierank@obe.tv>
    Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
    60fcc19b
Name
Last commit
Last update
..
aiff Loading commit data...
alaw Loading commit data...
asf Loading commit data...
ast Loading commit data...
au Loading commit data...
avi Loading commit data...
bmp Loading commit data...
caf Loading commit data...
dpx Loading commit data...
dv_fmt Loading commit data...
ffm Loading commit data...
flm Loading commit data...
flv_fmt Loading commit data...
gif Loading commit data...
gxf Loading commit data...
ircam Loading commit data...
ismv Loading commit data...
jpg Loading commit data...
mkv Loading commit data...
mmf Loading commit data...
mov Loading commit data...
mpg Loading commit data...
mulaw Loading commit data...
mxf Loading commit data...
mxf_d10 Loading commit data...
nut Loading commit data...
ogg Loading commit data...
pam Loading commit data...
pbmpipe Loading commit data...
pcx Loading commit data...
pgm Loading commit data...
pgmpipe Loading commit data...
pixfmt Loading commit data...
png Loading commit data...
ppm Loading commit data...
ppmpipe Loading commit data...
rm Loading commit data...
rso Loading commit data...
sgi Loading commit data...
smjpeg Loading commit data...
sox Loading commit data...
sunrast Loading commit data...
swf Loading commit data...
tga Loading commit data...
tiff Loading commit data...
ts Loading commit data...
voc Loading commit data...
voc_s16 Loading commit data...
w64 Loading commit data...
wav Loading commit data...
wtv Loading commit data...
xbm Loading commit data...
xwd Loading commit data...
yuv4mpeg Loading commit data...