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

HDR info #1

Closed
foXaCe opened this issue Jun 10, 2018 · 4 comments
Closed

HDR info #1

foXaCe opened this issue Jun 10, 2018 · 4 comments

Comments

@foXaCe
Copy link

foXaCe commented Jun 10, 2018

thank for your job.

it possible to add auto Max CLL, MAX FALL and Master-Display info like x265 codec ?

@Revan654
Copy link
Owner

Max-CLL and MAX-FALL can be added easily. However Master-Display is Tricky, Since allot of times the Master-Display settings is not known. Right now it just reads The single line, Which reads BT.2020. It needs all the data to import the data. If the actual coordinates are displayed it will import correctly.

Everything is Tested using MediaInfo 18.05. There been many Changes to HDR data and how it reads it. The old version of Mediainfo displays things differently.

@Revan654
Copy link
Owner

Some of the code looks off, I need to take a closer look at this and try to figure out the formula. It seems the White Point is creating the issue.

@Revan654
Copy link
Owner

Revan654 commented Jun 11, 2018

After looking through Mediainfo. There two options:.

  1. It it reads BT.2020 + MaxCLL/MaxFALL and loads max Fall and CLL normally. With Master-Display I can alter the code and it will just use the default Rec.2020. Since Mediainfo does not do the coordinates anymore. It was removed a few versions back. I tried to get them to re-add it but doesn't look like that is going to happen.
  2. Just have it load Max-CLL and Max-Fall and leave Master-Display alone. Which will require you to add in that data manually.

@Revan654
Copy link
Owner

Code has been altered, Will be seen in the next Release.

Repository owner locked as resolved and limited conversation to collaborators Nov 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants