Talk:Color presets
From Avisynth wiki
Revision as of 02:02, 14 December 2015 by Raffriff42 (Talk | contribs)
@Raffriff42,
I don't know what formula's you used to calculated the HSL values but they seem to be wrong. Wikipedia contains the formula's where 0<=H<360.0, 0<=S<=1 and 0<=L<=255. I suggest you use those. Example (color_blueviolet):
R = 138
G = 043
B = 226
M = max(R,G,B) = 226
m = min(R,G,B) = 43
C = M - m = 183
B = M => H = 60*(4+(R-G)/C) = 60*(4+(138-43)/183) = 271.15
L = (M+m)/2 = 134.5
S = C/(255-|2*L-255|) = 183/(255-|2*134.5-255|) = 0.76
Also you should add an appropriate reference for this. Admin 00:18, 19 August 2014 (CEST)
- Sure thing, I created RGB BlankClips with the values from colors_rgb.avsi, converted to YV12 with matrix="PC.601", and calculated hue as
- atan2(VPlaneMedian-128, UPlaneMedian-128)*180/PI + 360) % 360
- My complete script is posted here: [1] Raffriff42 15:31, 19 August 2014 (CEST)
- The values you posted stand for hue, saturation and luma derived of the YCbCr colorspace (you should have converted to yv24 instead). I suggest you name it (HSL)_ycbcr or something like that. Since the colorpresets are given in rgb values it would make more sense to include (HSL)_rgb instead. Admin 16:04, 19 August 2014 (CEST)
- Added a boxed note. What do you think? Raffriff42 19:57, 19 August 2014 (CEST)
- The hue and saturation values you give are a YCbCr concept while the color presets are given as RGB. I suggest you add the HSV and HSL values instead. That will be more useful to people. Admin 23:18, 21 August 2014 (CEST)
- I have made the requested changes. Raffriff42 08:32, 22 August 2014 (CEST)