Skip to content

Commit

Permalink
Update pipeline subdirectories in response to RFC-775.
Browse files Browse the repository at this point in the history
  • Loading branch information
erinleighh committed Jan 29, 2024
1 parent 5bc294f commit eeaa9ff
Show file tree
Hide file tree
Showing 7 changed files with 10 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ imports:
# a metrics subset because it would require constant micromanagement.
exclude:
- apPipe
- location: $AP_PIPE_DIR/pipelines/DarkEnergyCamera/ApPipe.yaml
- location: $AP_PIPE_DIR/pipelines/DECam/ApPipe.yaml
tasks:
# ApVerify override removed by excluding apPipe.
diaPipe:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ imports:
# a metrics subset because it would require constant micromanagement.
exclude:
- apPipe
- location: $AP_PIPE_DIR/pipelines/DarkEnergyCamera/ApPipeCalibrateImage.yaml
- location: $AP_PIPE_DIR/pipelines/DECam/ApPipeCalibrateImage.yaml
tasks:
# ApVerify override removed by excluding apPipe.
diaPipe:
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Verification pipeline specialized for DECam with fake source inejection.
# This concatenates various lsst.verify metrics to an AP pipeline

# This pipeline assumes $AP_PIPE_DIR/pipelines/DarkEnergyCamera/RunIsrForCrosstalkSources.yaml
# This pipeline assumes $AP_PIPE_DIR/pipelines/DECam/RunIsrForCrosstalkSources.yaml
# has already been run.

instrument: lsst.obs.decam.DarkEnergyCamera
Expand All @@ -10,7 +10,7 @@ imports:
- location: $AP_VERIFY_DIR/pipelines/_ingredients/ApVerifyWithFakes.yaml
exclude:
- processCcd
- location: $AP_PIPE_DIR/pipelines/DarkEnergyCamera/ProcessCcd.yaml
# Can't use $AP_PIPE_DIR/pipelines/DarkEnergyCamera/ApPipeWithFakes.yaml here
- location: $AP_PIPE_DIR/pipelines/DECam/ProcessCcd.yaml
# Can't use $AP_PIPE_DIR/pipelines/DECam/ApPipeWithFakes.yaml here
# because the changes made by that file and _ingredients/ApVerifyWithFakes.yaml
# are hard to separate.
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ imports:
# a metrics subset because it would require constant micromanagement.
exclude:
- apPipe
- location: $AP_PIPE_DIR/pipelines/HyperSuprimeCam/ApPipe.yaml
- location: $AP_PIPE_DIR/pipelines/HSC/ApPipe.yaml
tasks:
# ApVerify override removed by excluding apPipe.
diaPipe:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ imports:
# a metrics subset because it would require constant micromanagement.
exclude:
- apPipe
- location: $AP_PIPE_DIR/pipelines/HyperSuprimeCam/ApPipeCalibrateImage.yaml
- location: $AP_PIPE_DIR/pipelines/HSC/ApPipeCalibrateImage.yaml
tasks:
# ApVerify override removed by excluding apPipe.
diaPipe:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ imports:
- location: $AP_VERIFY_DIR/pipelines/_ingredients/ApVerifyWithFakes.yaml
exclude:
- processCcd
- location: $AP_PIPE_DIR/pipelines/HyperSuprimeCam/ProcessCcd.yaml
# Can't use $AP_PIPE_DIR/pipelines/HyperSuprimeCam/ApPipeWithFakes.yaml here
- location: $AP_PIPE_DIR/pipelines/HSC/ProcessCcd.yaml
# Can't use $AP_PIPE_DIR/pipelines/HSC/ApPipeWithFakes.yaml here
# because the changes made by that file and _ingredients/ApVerifyWithFakes.yaml
# are hard to separate.
2 changes: 1 addition & 1 deletion python/lsst/ap/verify/pipeline_driver.py
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ def runApPipeGen3(workspace, parsedCmdLine, processes=1):
# processing pipeline (DM-31492).
instruments = {id["instrument"] for id in workspace.workButler.registry.queryDataIds("instrument")}
if "DECam" in instruments:
crosstalkPipeline = "${AP_PIPE_DIR}/pipelines/DarkEnergyCamera/RunIsrForCrosstalkSources.yaml"
crosstalkPipeline = "${AP_PIPE_DIR}/pipelines/DECam/RunIsrForCrosstalkSources.yaml"
crosstalkArgs = ["pipetask", "run",
"--butler-config", workspace.repo,
"--pipeline", crosstalkPipeline,
Expand Down

0 comments on commit eeaa9ff

Please sign in to comment.