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
DM-10957: Add measurement algorithm for point sources on fluff #107
Conversation
src/LocalBackground.cc
Outdated
assert(imageValues.getNumElements() == maskValues.getNumElements()); // constructed from the same spans | ||
auto maskIter = maskValues.begin(); | ||
for (auto imageIter = imageValues.begin(); imageIter != imageValues.end(); ++imageIter, ++maskIter) { | ||
if (*maskIter & maskVal) continue; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a better name for maskVal
, to distinguish it from maskValues
? ignoredPixelBitmask
or badPixelBitmask
?
I think the comparison reads a little easier if its if (...) != 0 continue;
. Or
if (... == 0) {
values.push_back(...)
}
LSST_CONTROL_FIELD(annulusOuter, float, | ||
"Outer radius for background annulus as a multiple of the PSF sigma"); | ||
LSST_CONTROL_FIELD(bgRej, float, | ||
"Rejection threshold for background measurement"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mention this is in units of standard deviations?
LSST_CONTROL_FIELD(bgRej, float, | ||
"Rejection threshold for background measurement"); | ||
LSST_CONTROL_FIELD(bgIter, int, | ||
"Number of iterations for background measurement"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
number of sigma-clipping iterations.
// -*- lsst-c++ -*- | ||
/* | ||
* LSST Data Management System | ||
* Copyright 2008-2017 AURA/LSST. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might as well say 2018 in the boilerplates, fwiw.
Useful for subtracting a revised background (per pixel) estimate.
Some sources are superimposed on others, and so measurement of a small-scale background is helpful for getting the true flux measurement.
For both single-frame and forced measurement modes.
922cc3d
to
ac081cf
Compare
No description provided.