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.

Sign up for GitHub

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

Jump to bottom

DM-43196: Fix Fakes pipelines to point to correct ProcessCcd #215

Merged

parejkoj merged 1 commit into main from tickets/DM-43196

Mar 7, 2024

Contributor

parejkoj commented Mar 6, 2024

{Summary of changes. Prefix PR title with JIRA issue.}

Do unit tests pass (scons and/or stack-os-matrix)?
Did you run ap_verify.py on at least one of the standard datasets?
For changes to metrics, the print_metricvalues script from lsst.verify will be useful.
Is the Sphinx documentation up-to-date?

kfindeisen approved these changes

View reviewed changes

pipelines/DECam/ApVerifyWithFakes.yaml

@@ @@ -10,7 +10,7 @@ imports: @@
                 - location: $AP_VERIFY_DIR/pipelines/_ingredients/ApVerifyWithFakes.yaml
                   exclude:
                     - processCcd
-                - location: $AP_PIPE_DIR/pipelines/DECam/ProcessCcd.yaml
+                - location: $AP_PIPE_DIR/pipelines/DECam/ProcessCcdCalibrate.yaml

Member

kfindeisen Mar 6, 2024

Same comments as in ap_pipe?


          Fix Fakes pipelines to point to correct ProcessCcd

528559d

parejkoj force-pushed the tickets/DM-43196 branch from 7f89515 to 528559d Compare

March 7, 2024 00:27

parejkoj merged commit 3679153 into main

2 checks passed

parejkoj deleted the tickets/DM-43196 branch

March 7, 2024 00:38

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment