Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#13419 closed Bug Report - General (fixed)

Jerky playback with shield on certain interlaced content

Reported by: mspieth Owned by: Peter Bennett
Priority: minor Milestone: 30.1
Component: Ports - Android Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

Some recordings are giving extremely jerky playback.

The log shows that the video is being re-initialized around twice every second.

Attachments (2)

20190303_1520_video_reinit_bug.patch (770 bytes ) - added by Peter Bennett 7 years ago.
Fix for repeated video reinitialization bug
20190304_1720_video_reinit_bug.patch (586 bytes ) - added by Peter Bennett 7 years ago.
Replacement for the previous patch. Hopefully this fixes it for all recordings.

Download all attachments as: .zip

Change History (8)

by Peter Bennett, 7 years ago

Fix for repeated video reinitialization bug

comment:1 by Peter Bennett, 7 years ago

Reporter: changed from Peter Bennett to mspieth
Status: assignedaccepted

The logic is continuously flipping between 25fps and 50fps and each time is re-initializing video.

The fix changes the order of fields used to determine framerate to give r_frame_rate a lower priority. r_frame_rate is described as the lowest possible frame rate. not the actual frame rate and was showing as 50. I think the description actually means highest possible frame rate, lowest frame interval

This fixes playback for Mark's sample file.

comment:2 by mspieth, 7 years ago

Tested with another 2 programs which is still very jerky. Tried at 1.0 and 1.5x.

1013_20190303003000.ts (another sports 6.5h)

also another style of program

1013_20190303100000.ts (chat show style) 1h

by Peter Bennett, 7 years ago

Replacement for the previous patch. Hopefully this fixes it for all recordings.

comment:3 by Peter Bennett, 7 years ago

This bug actually manifests on all decoders that have built in deinterlace - medicodec, nvdec, vaapi2. It also manifests on avsync as well as avsync2.

The new patch fixes it on the original video as well as the two new videos from Mark.

comment:4 by mspieth, 7 years ago

Looks good so far.

comment:5 by Peter Bennett <pbennett@…>, 7 years ago

Resolution: fixed
Status: acceptedclosed

In 0e7e57f9c2/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available

comment:6 by Peter Bennett <pbennett@…>, 7 years ago

In 76ea9755b/mythtv:

Error: Processor CommitTicketReference failed
GIT backend not available
Note: See TracTickets for help on using tickets.