Skip to content

Commit

Permalink
Do not catch all errors for wfau tests
Browse files Browse the repository at this point in the history
  • Loading branch information
robertdstein committed Jul 10, 2023
1 parent 9c5ce39 commit 6834176
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
4 changes: 3 additions & 1 deletion mirar/pipelines/winter/build_references.py
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,7 @@ def run_winter_reference_build_pipeline(
full_dec_size_deg: float = 1.2,
field_id: int | None = None,
subdet_id: int | None = None,
catch_all_errors: bool = True,
):
"""
Run the reference build pipeline on the winter fields
Expand All @@ -174,6 +175,7 @@ def run_winter_reference_build_pipeline(
full_dec_size_deg: Full declination size of the field in degrees
field_id: Run only for this fieldid (for debugging)
subdet_id: Run only for this subdetid (for debugging)
catch_all_errors: Catch all errors and continue
Returns:
"""
Expand Down Expand Up @@ -219,7 +221,7 @@ def run_winter_reference_build_pipeline(

res, errorstack = pipeline.reduce_images(
dataset=dataset,
catch_all_errors=True,
catch_all_errors=catch_all_errors,
)

if len(res) == 0:
Expand Down
2 changes: 1 addition & 1 deletion tests/test_wfau_references.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ def test_pipeline(self):
self.logger.info("\n\n Testing WINTER reference building pipeline \n\n")

res, _ = run_winter_reference_build_pipeline(
subdet_id=0, field_id=TEST_WINTER_FIELD_ID
subdet_id=0, field_id=TEST_WINTER_FIELD_ID, catch_all_errors=False
)

self.assertEqual(len(res[0]), 1)
Expand Down

0 comments on commit 6834176

Please sign in to comment.