Modify

Ticket #4637 (closed patch: fixed)

Opened 4 years ago

Last modified 3 years ago

Allow a little leeway before rejecting double rate deinterlacers (bob/greedyh/yadif)

Reported by: mark.kendall@… Owned by: ijr
Priority: minor Milestone: unknown
Component: Video Playback Version: unknown
Severity: medium Keywords: easy
Cc: Ticket locked: no

Description

In summary, the nvidia drivers increase the htotal of the accepted modeline to ensure it is divisible by 8. This changes the calculated refresh rate and can lower it below the 2* framerate threshold needed for bob etc.

The attached patch adds a little tolerance to the accept/reject decision to ensure the best deinterlacing options can still be used.

I'm seeing this on several frontends where the EDID reported timings for 50Hz modelines give a refresh rate of 50Hz exactly, but by the time the drivers have adjusted the modeline the refresh rate drops to circa 49.9Hz.

Attachments

vsync.diff (1.1 KB) - added by anonymous 4 years ago.

Change History

Changed 4 years ago by anonymous

comment:1 Changed 4 years ago by laga

  • Keywords easy added
  • Component changed from mythtv to Video Playback

comment:2 Changed 3 years ago by markk

  • Status changed from new to closed
  • Resolution set to fixed

(In [19287]) Allow a small amount of tolerance before rejecting double rate deinterlacers due to low refresh rates. This ensures bobdeint etc will still work when the video driver (specifically the nvidia proprietary driver) adjusts the htotal to ensure it is divisble by 8. Closes #4637

comment:3 Changed 3 years ago by thomas@…

I applied [19287] to 0.21 fixes and it works there, too. Please backport.

View

Add a comment

Modify Ticket

Action
as closed
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.