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-24738: Add background offset tracking to fgcmcal #27
Conversation
Add delta-aperture background epsilon offset computations.
Update delta_aper epsilon to compute global backgrounds in better location.
Fix multprocessing on macos in new delta aper code.
Add default telescope rotation and fix index bug for delta-aper computation.
self.fgcmLog.info("Cannot compute background x/y correlations without x/y information") | ||
return | ||
|
||
from .fgcmUtilities import plotCCDMapBinned2d |
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 reason why the import is happening here, and not at the top of the file?
Fix delta aper units and other delta aper plotting bugs.
@@ -63,7 +63,7 @@ def __init__(self, fgcmConfig, fgcmPars, fgcmStars): | |||
self.fgcmLog.warn('No aperture radii set. Epsilon is unnormalized.') | |||
self.epsilonNormalized = False | |||
|
|||
self.njyZp = 48.6 - 9*2.5 | |||
self.njyZp = 8.9 + 9*2.5 |
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.
I know I'm not technically required to review this PR, but woah, that's a huge drop. What happened here?
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.
I got my units confused completely on cgs vs janskys.
Add option to force htm matching when making stars.
This adds the ability to delta-aperture-magnitude to track background offsets. Also some deprecation warnings have been fixed.