Ticket #7759 (closed patch: fixed)
Opened 3 years ago
Last modified 3 years ago
[PATCH] Add support for repeat_pict plus related AV-sync improvements
|Reported by:||tralph11@…||Owned by:||markk|
|Component:||MythTV - Video Playback||Version:||unknown|
I've created a patch that implements the correct video delays when frames with the repeat_pict flag are processed. This dramatically improves playback for HD 720p and 1080i broadcast material. Also included are some AV-sync improvements which improve next trigger management. I'm including a complete patch and also in parts to allow ease of integration and discussion.
- Part 1
This implements the repeat_pict delay logic in NVP and also increases the allowed delay by CalcDelay? in vsync.cpp
- Part 2
The next trigger was being allowed to fall behind too far when trying to speed up video playback faster than possible. This caused severe playback issues when repeated frames were encountered or a frame dropped.
- Part 3
This prevents the trigger from being advanced when a frame is dropped. If the frame is dropped but the next trigger is advanced you lose the benefit of dropping the frame.
This set of patches fully addresses all the problems I've seen in the past and also resolves some extreme cases reported in the following discussion:
comment:1 Changed 3 years ago by tralph11@…
comment:2 Changed 3 years ago by markk
- Owner changed from janne to markk
- Status changed from new to accepted
comment:3 Changed 3 years ago by markk
- Milestone changed from unknown to 0.23
comment:4 Changed 3 years ago by markk
- Status changed from accepted to closed
- Resolution set to fixed