-
-
Notifications
You must be signed in to change notification settings - Fork 56
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
Default convolution mode introduces edge effects #8
Comments
So yeah, develop builds, not sure what is happening in your branch, I will have to pull it down. |
There is probably something silly wrong in the tests I wrote. I haven't yet been able to run the test suite myself as I'm having trouble getting OpenImageIO to work. However, I also think that the reason the existing tests are failing because they are testing against images that have this exact issue (demosaiced using the reflected padding). I'm actually wondering whether any demosaicing codes deal with this issue. |
The default image extension mode used in
scipy.ndimage.convolve
is'reflect'
, which reflects the image about the edge of the pixels. This introduces artefacts in the edge of the resulting RGB layers when demosaicing:Passing
mode='mirror'
- which reflects about pixel centers (doesn't repeat the edge values) - toconvolve
indemosaicing_CFA_Bayer_bilinear
returns the expected values. I figure this should be the default for demosaicing?The text was updated successfully, but these errors were encountered: