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-37141: Switch to gbdesAstrometricFit #53
Conversation
cc57fe0
to
917a652
Compare
@@ -19,6 +19,8 @@ tasks: | |||
class: lsst.jointcal.JointcalTask | |||
config: | |||
connections.inputSourceTableVisit: preSourceTable_visit | |||
gbdesAstrometricFit: |
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.
Remove jointcal too. DRP-full.yaml should be a complete pipeline without redundant leaves that could in theory run from beginning to end (like in ci_hsc_gen3)
@@ -44,6 +44,7 @@ tasks: | |||
photo_calib_provider: "tract" | |||
background_provider: "input_summary" | |||
connections.photoCalibName: "jointcal" | |||
connections.skyWcsName: "jointcal" |
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.
Up in line 13, I think you should exclude gbdesAstrometricFit
along with fgcm.
@@ -52,6 +53,7 @@ tasks: | |||
modelPsf.defaultFwhm: 7.7 | |||
warpAndPsfMatch.warp.warpingKernelName: lanczos5 | |||
coaddPsf.warpingKernelName: lanczos5 | |||
connections.skyWcsName: jointcal |
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.
and if you remove jointcal from DRP-Full.yaml, it needs to go here for DECam.
class: lsst.jointcal.JointcalTask
config:
connections.inputSourceTableVisit: preSourceTable_visit
917a652
to
edb61a2
Compare
No description provided.