Add autodilation for blends. #236
Conversation
jbkalmbach
force-pushed
the
tickets/DM-38557_2
branch
from
3926652
to
f230e54
Compare
|
Closed Loop Results showing that auto dilate performs at least as well as or better than no dilation:
|
python/lsst/ts/wep/imageMapper.py
Outdated
| """ | ||
| sourceMaskInit = sourceMask.copy() | ||
| blendMaskInit = blendMask.copy() | ||
| imageArray = image.image |
There was a problem hiding this comment.
Please consider do not have this new variable imageArray or use the image.image.copy() instead.
Since this name is not too long, the followings could be used directly:
blendPix = newSourceMask * image.image
Otherwise, someone might not notice this in the future and modify the value in imageArray, and therefore, touch the original image.image by the accident.
When this happened, it would not be easy to debug.
Thanks!
I'm not sure what to make of these plots. What is the difference between the two? Why is autodilate underperforming compared to dilate=4? |
No description provided.