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

Question about constant terms C1, C2. #26

Open
SaulZhang opened this issue Nov 16, 2019 · 2 comments
Open

Question about constant terms C1, C2. #26

SaulZhang opened this issue Nov 16, 2019 · 2 comments

Comments

@SaulZhang
Copy link

From the original paper, C1=(KL)², but I didn't see L=255 just K1 = 0.01 in the specific implementation of the code. Can the item L be discarded?Does the code mean K1=0.01?
image

@LiUzHiAn
Copy link

LiUzHiAn commented Mar 4, 2020

I have the same question. If my images are normalized into [-1,1] or [0,1], should I have to modify the c_1 and c_2 constant?

@SaulZhang, How about your use situation? Thx~

@LiUzHiAn
Copy link

LiUzHiAn commented Mar 4, 2020

@SaulZhang Well, I find another multi-scale SSIM repo implementation, which is based on this repo exactly. There, the variable L is taken into consideration. For the 0-255 gray image, L is set to be 255. For [-1,1] range, L is set to be 2.

Link is: https://github.com/jorge-pessoa/pytorch-msssim/blob/master/pytorch_msssim/init.py. Check it out !

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