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-28107: Convert/ingest PS1 and Gaia refcats into gen3 repo for DECam use #69
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.
Handful of documentation suggestions.
@@ -0,0 +1,67 @@ | |||
description: The AP template building pipeline specialized for DECam |
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.
Period at end of sentence?
Is this designed for a specific dataset? If so, would be worth mentioning that 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.
It's not designed for a specific dataset, it should work for anything DECam. It should be easy to adapt for other cameras, some of the configs (like isr.doDark!) may need to change.
doDefect: True | ||
doNanMasking: True | ||
doInterpolate: True | ||
doDark: False |
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.
Why no darks?
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.
DECam doesn't have darks
photoCal.match.referenceSelection.magLimit.fluxField: 'i_flux' | ||
photoCal.match.referenceSelection.magLimit.maximum: 22.0 | ||
astromRefObjLoader.ref_dataset_name: 'gaia' | ||
#astromRefObjLoader.anyFilterMapsToThis: 'phot_g_mean' # not yet implemented in gen3, DM-27843 |
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.
Thanks for referencing this ticket.
pipelines/APTemplate.yaml
Outdated
config: | ||
doApplySkyCorr: False | ||
#doApplyExternalPhotoCalib: False # not yet implemented in gen3 anyway | ||
#doApplyExternalSkyWcs: False # not yet implemented in gen3 anyway |
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.
The "anyway"s here confuse me.
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.
We usually have these turned off when building templates, and couldn't turn them on if we wanted to, anyway.
dc65f41
to
4241a02
Compare
This ticket adds a starter gen3 pipeline for building coadds (for use as diffim templates) from scratch.