Skip to content
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-27613: Add gen3 ability to add external SkyWcs and PhotoCalib #475

Merged
merged 1 commit into from Mar 3, 2021

Conversation

erykoff
Copy link
Contributor

@erykoff erykoff commented Mar 1, 2021

No description provided.

Copy link
Contributor

@fred3m fred3m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine to me (after it is rebased), just make sure to resolve the questions raised by @parejkoj .

python/lsst/pipe/tasks/coaddBase.py Show resolved Hide resolved
dtype=bool,
default=True,
doc=("When using doApplyExternalPhotoCalib, use global calibrations. "
"When False, per-tract calibrations will be used instead.")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should the doc be a bit more explicit as to what is meant by global calibrations (FGCM for photometry and jointcal for astrometry), or will this be clear to people who need to worry about this?

python/lsst/pipe/tasks/makeCoaddTempExp.py Show resolved Hide resolved
@erykoff erykoff merged commit 880d6fa into master Mar 3, 2021
@erykoff erykoff deleted the tickets/DM-27613 branch March 3, 2021 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants