-
Notifications
You must be signed in to change notification settings - Fork 17
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-12445: Set appropriate default configs for CompareWarp Coadds #97
Conversation
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.
A few things to fix up.
config/makeCoaddTempExp.py
Outdated
config.makePsfMatched=True | ||
config.warpAndPsfMatch.psfMatch.kernel['AL'].kernelSize=29 | ||
config.warpAndPsfMatch.psfMatch.kernel['AL'].alardSigGauss=[1.0, 2.0, 4.5] | ||
config.modelPsf.defaultFwhm=7.7 |
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.
Spaces either side of the =
please.
config/coaddDriver.py
Outdated
config.assembleCoadd.retarget(CompareWarpAssembleCoaddTask) | ||
config.assembleCoadd.assembleStaticSkyModel.subregionSize = (10000, 200) | ||
config.assembleCoadd.assembleStaticSkyModel.badMaskPlanes = ("BAD", "EDGE", "SAT", "INTRP", "NO_DATA",) | ||
config.assembleCoadd.assembleStaticSkyModel.doApplyUberCal = True |
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.
To eliminate code duplication, this should load the configuration from the configuration file for compareWarpAssembleCoadd (like below). And I think the additional loading below may potentially clobber what you've got 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.
Agreed. Note that by that reasoning it currently should be reading in safeClipAssembleCoadd.py
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 think https://jira.lsstcorp.org/browse/DM-10634 might be relevant here.
a0d7b8c
to
3f3b646
Compare
including making psfMatched Warps by default.
3f3b646
to
4c49300
Compare
No description provided.