-
Notifications
You must be signed in to change notification settings - Fork 47
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
Implementation different from RAISR paper? #1
Comments
Dear Linh, Thank you for sharing your analysis of our RAISR implementation. Your observations are correct. The following are the responses to your questions:
Thank you. With regards, |
Hi, Thanks for your detailed response. Regarding (1), my question was that Blending1() is not using LCC size, but how many neighbors have lower values than the center pixel: 0 | 1 | 1 1 | 0 | 0 I'm actually not sure how the paper determines connected components in a CT. So please correct me if I'm wrong. |
Hi,
Thanks for your great work. I noticed some (apparent) differences between your implementation and the RAISR paper:
Functions.py at commit ae62ae5: function Blending1() scales HR pixel weight with the number of neighboring pixels that have smaller values, not the LCC size. Both are 3 in RAISR Fig 9, but not necessarily in all cases.
Test.py at commit b533df3, line 169-174: It seems you're copying the Cb and Cr channel from the ground truth HR image to result_ycbcr. According to RAISR section VI, the chromatic channels should be upscaled by bicubic interpolation from the LR image.
Am I reading your code correctly, and are these discrepancies intentional?
Thanks!
The text was updated successfully, but these errors were encountered: