Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1080p videos take 1080i settings #96

Closed
mighty-p opened this issue Mar 11, 2018 · 8 comments
Closed

1080p videos take 1080i settings #96

mighty-p opened this issue Mar 11, 2018 · 8 comments

Comments

@mighty-p
Copy link

Sorry for filing so many issues at the moment. I thought it is good filing everything that I noticed. This should be the last one for the moment :)

I noticed that for DVB-T2 channels, which are sent here in 1080p resolution, the picture is influenced by the settings that I do for 1080i. I was surprised that the settings for an interlaced resolution are actually applied here for a progressive one (even if the number of lines are the same). Actually, I thought the UHD settings would apply, although 1080p is also not UHD, so this would also not be the best fit. Maybe there should be separate settings for 1080p, or alternatively the 1080i section should be renamed to "1080i/1080p".

@rofafor
Copy link
Contributor

rofafor commented Mar 12, 2018

Could you check whether the linked pull requests fixes your problems? I don't have 1080p samples available, so I cannot check the current functionality of interlace detection.

@mighty-p
Copy link
Author

Unfortunately, I am currently not at the location with my vdr with DVB-T2 reception. I probably won't be able to test this before Saturday.

@mighty-p
Copy link
Author

Unfortunately, for me even with this change, 1080p takes the 1080i settings, not the 1080i ones.

@mighty-p
Copy link
Author

not sure why I cannot re-open this issue.

@rofafor
Copy link
Contributor

rofafor commented Mar 16, 2018

Please, provide a short video sample.

@mighty-p
Copy link
Author

00001.ts.gz

@rofafor
Copy link
Contributor

rofafor commented Mar 17, 2018

This might help: #112

@mighty-p
Copy link
Author

I confirm that this helped. Thanks a lot!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants