Skip to content

Commit

Permalink
DM-47906 fix to pass processing_type to different jobs
Browse files Browse the repository at this point in the history
  • Loading branch information
zhaoyu committed Dec 20, 2024
1 parent 28d841b commit ba07eb2
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 1 deletion.
1 change: 1 addition & 0 deletions doc/changes/DM-47906.fix.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
fix to pass processing_type to different jobs
6 changes: 5 additions & 1 deletion python/lsst/ctrl/bps/panda/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -250,9 +250,13 @@ def _make_doma_work(
cvals = {"curr_cluster": gwjob.label}
_, site = config.search("computeSite", opt={"curvals": cvals, "required": True})
cvals["curr_site"] = site
cvals["curr_pipetask"] = gwjob.label

Check warning on line 253 in python/lsst/ctrl/bps/panda/utils.py

View check run for this annotation

Codecov / codecov/patch

python/lsst/ctrl/bps/panda/utils.py#L253

Added line #L253 was not covered by tests
_, processing_type = config.search(
"processing_type", opt={"curvals": cvals, "default": PANDA_DEFAULT_PROCESSING_TYPE}
"processingType", opt={"curvals": cvals, "default": PANDA_DEFAULT_PROCESSING_TYPE}
)
if gwjob.label == "finalJob":
_, final_job = config.search(gwjob.label)
processing_type = final_job["processingType"] if final_job["processingType"] else "merge"

Check warning on line 259 in python/lsst/ctrl/bps/panda/utils.py

View check run for this annotation

Codecov / codecov/patch

python/lsst/ctrl/bps/panda/utils.py#L258-L259

Added lines #L258 - L259 were not covered by tests
_, task_type = config.search("taskType", opt={"curvals": cvals, "default": PANDA_DEFAULT_TASK_TYPE})
_, prod_source_label = config.search(
"prodSourceLabel", opt={"curvals": cvals, "default": PANDA_DEFAULT_PROD_SOURCE_LABEL}
Expand Down

0 comments on commit ba07eb2

Please sign in to comment.