python-colormath icon indicating copy to clipboard operation
python-colormath copied to clipboard

de2000中有错误

Open vip594nb opened this issue 5 years ago • 4 comments

def delta_e_cie2000(lab_color_vector, lab_color_matrix, Kl=1, Kc=1, Kh=1): R_C = numpy.sqrt((numpy.power(avg_C1p_C2p, 7.0)) / (numpy.power(avg_C1p_C2p, 7.0) + numpy.power(25.0, 7.0))) 应该为 R_C = 2*(numpy.sqrt((numpy.power(avg_C1p_C2p, 7.0)) / (numpy.power(avg_C1p_C2p, 7.0) + numpy.power(25.0, 7.0)))) 少乘以2了

vip594nb avatar Jan 30 '20 15:01 vip594nb

Hi @vip594nb,

Would it be possible to update/re-post in english please?

Cheers,

Thomas

KelSolaar avatar Jan 31 '20 09:01 KelSolaar

It says

def delta_e_cie2000(lab_color_vector, lab_color_matrix, Kl=1, Kc=1, Kh=1): R_C = numpy.sqrt((numpy.power(avg_C1p_C2p, 7.0)) / (numpy.power(avg_C1p_C2p, 7.0) + numpy.power(25.0, 7.0))) should be R_C = 2*(numpy.sqrt((numpy.power(avg_C1p_C2p, 7.0)) / (numpy.power(avg_C1p_C2p, 7.0) + numpy.power(25.0, 7.0)))) (multiply by 2)

JensBloemer avatar Mar 13 '20 09:03 JensBloemer

I don't think this is a mistake. The 2 in this case is contained in R_T. R_T = -2 * R_C * numpy.sin(2 * numpy.radians(delta_ro))

So, R_C = numpy.sqrt((numpy.power(avg_C1p_C2p, 7.0)) / (numpy.power(avg_C1p_C2p, 7.0) + numpy.power(25.0, 7.0))) is OK.

s51517765 avatar Sep 01 '21 07:09 s51517765

@vip594nb @KelSolaar I have checked. It all matches (4 decimal places) with TABLE 1. of the following paper, and I think it is correct. http://www2.ece.rochester.edu/~gsharma/ciede2000/ciede2000noteCRNA.pdf

s51517765 avatar Sep 07 '21 08:09 s51517765