Fix Version/s: None
Team:Data Release Production
In smoothArray multiplying the convolved image by numerator/denominator, was supposed to correct the roll-off produced by padding the input image with zeros.
And conversation on #subaru_hsc:
Sogo Mineo [8:42 PM] These lines https://github.com/lsst/pipe_drivers/blob/master/python/lsst/pipe/drivers/background.py#L869
should be fixed (naively) as:
Let me explain: we set bad pixels to 0 in computing convolved. Doing it wrongly lowers the pixels around the bad pixels in convolved . We compensate this effect with *numerator/denominator. In computing convolved , pixels outside the input image are also set to 0 ( cval=0.0 ). It is that these pixels are regarded as a kind of bad pixels, and their effect should have been compensated together with the effect of the bad pixels inside the input image. This is why the numerator needs the fix above.
But if we compute numerator according to this fix, all elements of numerator are 1. Therefore, we can drop it altogether, hence Koike-san's proposal:
|Status||To Do [ 10001 ]||In Progress [ 3 ]|
|Status||In Progress [ 3 ]||Reviewed [ 10101 ]|
|Resolution||Done [ 10000 ]|
|Status||Reviewed [ 10101 ]||Done [ 10002 ]|