DM-49059: Stand-up an obs_decam calibrateImage config script#251
DM-49059: Stand-up an obs_decam calibrateImage config script#251leeskelvin merged 1 commit intomainfrom
Conversation
leeskelvin
force-pushed
the
tickets/DM-49059
branch
from
5febfb8 to
f3753a2
Compare
config/calibrateImage.py
Outdated
| config.astrometry_ref_loader.load(os.path.join(config_dir, "filterMap.py")) | ||
| # Use the filterMap instead of the "any" filter (as is used for Gaia) | ||
| config.astrometry_ref_loader.anyFilterMapsToThis = None |
There was a problem hiding this comment.
No filterMap for Gaia: you want anyFilterMapsToThis = "phot_g_mean", which is the default anyway.
There was a problem hiding this comment.
Edited to phot_g_mean for clarity. Should I also remove the ref_loader filterMap.py line above?
There was a problem hiding this comment.
Yes: anyFilterMapsToThis and a filterMap are mutually exclusive (and should raise an error if both are set).
There was a problem hiding this comment.
Removed - much cleaner now, thanks. Tested to confirm this all works on DECam data here at Princeton.
config/calibrateImage.py
Outdated
| config_dir = os.path.dirname(__file__) | ||
|
|
||
| # Astrometry | ||
| config.connections.astrometry_ref_cat = "gaia_dr3_20230707" |
There was a problem hiding this comment.
I'd not include this, and the photometry_ref_cat below, since they're the default. If you are going to respecify the default, please put a comment as to why.
config/calibrateImage.py
Outdated
| config.photometry_ref_loader.load(os.path.join(config_dir, "filterMap.py")) | ||
|
|
||
| # Adopt HSC calibrateImage defaults while older defaults are compared | ||
| config.install_simple_psf.fwhm = 1.5 * 2.0 * math.sqrt(2.0 * math.log(2.0)) |
There was a problem hiding this comment.
Keep this if you want, but I'd try running it with the calibrateImage default and see. I don't think this HSC setting really translates to DECam, since they have different pixel scales and seeing.
config/calibrateImage.py
Outdated
|
|
||
| # Adopt HSC calibrateImage defaults while older defaults are compared | ||
| config.install_simple_psf.fwhm = 1.5 * 2.0 * math.sqrt(2.0 * math.log(2.0)) | ||
| config.psf_detection.thresholdType = "stdev" |
There was a problem hiding this comment.
DECam shouldn't have the same variance problem that HSC does, so I'd go with the default here.
leeskelvin
force-pushed
the
tickets/DM-49059
branch
from
f3753a2 to
638f0e4
Compare
leeskelvin
force-pushed
the
tickets/DM-49059
branch
from
638f0e4 to
3b73877
Compare
2 participants
No description provided.