From 3b067ed4ec292e9e7e5b8bdeb106514221cc0a50 Mon Sep 17 00:00:00 2001 From: erazortt Date: Sun, 30 Jan 2022 21:30:59 +0100 Subject: [PATCH 1/2] unify letter case of maxFall to MaxFALL --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index 542e7ed..764f49e 100644 --- a/README.md +++ b/README.md @@ -21,7 +21,7 @@ clip = clip.ConvertToRGB64(matrix="Rec2020") clip.MaxCLLFind() ``` -Supported are the packed RGB formats RGB24, RGB32, RGB48, RGB64 and the planar RGB formats RGBP8, RGBP10, RGBP12, RGBP14, RGBP16. It is not advisable to use the formats RGB24, RGB32 and RGBP8 for HDR clips. The planar formats are processed about 30% faster than the packed formats, however the default maxFall calculation is unsuppored for them. +Supported are the packed RGB formats RGB24, RGB32, RGB48, RGB64 and the planar RGB formats RGBP8, RGBP10, RGBP12, RGBP14, RGBP16. It is not advisable to use the formats RGB24, RGB32 and RGBP8 for HDR clips. The planar formats are processed about 30% faster than the packed formats, however the default MaxFALL calculation is unsuppored for them. ### Alternate MaxFALL algorithm The default MaxFALL algorithm uses the SMPTE recommendation of averaging max(R,G,B) across all pixels, meaning the brightest channel of each pixel goes into the average. If you want the average of all channels of all pixels (not the official recommendation) instead, do this: @@ -30,7 +30,7 @@ clip.MaxCLLFind(maxFallAlgorithm=1) ``` This is more for your own curiosity and might lead to playback problems like flickering if used as actual HDR metadata, since it typically leads to typically slightly lower average intensity readings and if the TV bases its own dimming on the official recommendation, it might dim the image when it reaches a higher FALL than your calculated MaxFALL, which will almost certainly happen. -The official maxFall algorithm is not supported by the planar formats. If your input is planar either disable the maxFall calculation or use the unofficial algorithm or convert your clip to a packed format like RGB48. The maxFall calculation is disabled by: +The official MaxFALL algorithm is not supported by the planar formats. If your input is planar either disable the MaxFALL calculation or use the unofficial algorithm or convert your clip to a packed format like RGB48. The MaxFALL calculation is disabled by: ``` clip.MaxCLLFind(maxFallAlgorithm=-1) From 5a1eba0ad4d61b96638b7552e8f9379bf7619bae Mon Sep 17 00:00:00 2001 From: erazortt Date: Thu, 10 Feb 2022 18:40:49 +0100 Subject: [PATCH 2/2] remove unnecessary warning in readme --- README.md | 3 --- 1 file changed, 3 deletions(-) diff --git a/README.md b/README.md index 764f49e..bade44b 100644 --- a/README.md +++ b/README.md @@ -4,9 +4,6 @@ PQ HDR Analyzer plugin for AVISynth, analyzes MaxCLL and MaxFALL and writes that The created textfile's name is "MaxCLLFind_Results0.txt". It will be overwritten if it already exists. -**Caution:** This may not be the correct way to calculate MaxFALL and MaxCLL. For (Max)FALL I averaged the nit intensities of every single channel of every pixel in the frame. For MaxCLL I simply took the brightest channel of any pixel in any of the frames. There may be some weighting necessary akin to what was done here: https://github.com/HDRWCG/HDRStaticMetadata - - ### Usage ``` clip.MaxCLLFind()