Skip to content

Commit

Permalink
Update astrometry fail tests
Browse files Browse the repository at this point in the history
calibrateImage needs a tighter constraint here to produce a failure,
and it's only in one detector/visit.
  • Loading branch information
parejkoj authored and TallJimbo committed Nov 7, 2024
1 parent c0b4a96 commit 4d3b23f
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 4 deletions.
2 changes: 1 addition & 1 deletion bin/pipeline.sh
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ pipetask --long-log --log-level="$loglevel" qgraph \
-b "$repo"/butler.yaml \
--input "$INPUTCOLL" --output "$COLLECTION" \
-p "$DRP_PIPE_DIR/pipelines/HSC/DRP-ci_hsc.yaml" \
-c calibrateImage:astrometry.maxMeanDistanceArcsec=0.025 \
-c calibrateImage:astrometry.maxMeanDistanceArcsec=0.020 \
-c makeWarp:select.maxPsfTraceRadiusDelta=0.2 \
--save-qgraph "$QGRAPH_FILE"

Expand Down
3 changes: 1 addition & 2 deletions python/lsst/ci/hsc/gen3/data.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,13 +54,12 @@
{'visit': 903988, 'detector': 24, 'physical_filter': 'HSC-I'},
]
# The following lists the dataIds that fail the astrometry check with
# the config override calibrate.astrometry.maxMeanDistanceArcsec=0.025
# the config override calibrate.astrometry.maxMeanDistanceArcsec=0.020
# set. This list is sensitive to the astrometry algorithms and dataset
# under consideration, so may require updating if either of those change
# in the context of this repository.
ASTROMETRY_FAILURE_DATA_IDS = [
{'visit': 903344, 'detector': 0, 'physical_filter': 'HSC-R'},
{'visit': 903346, 'detector': 1, 'physical_filter': 'HSC-R'},
]
# The following lists the dataIds that fail the PSF Model robustness check
# with the config override makeWarp.select.maxPsfTraceRadiusDelta=0.2 set.
Expand Down
2 changes: 1 addition & 1 deletion tests/test_astrometryFail.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ def setUp(self):
self.butler = Butler(os.path.join(getPackageDir("ci_hsc_gen3"), "DATA"), writeable=False,
collections=["HSC/calib/2013-06-17", "HSC/runs/ci_hsc"])
# The dataId here represents one of the astrometry fit failures
# imposed by setting astrometry.maxMeanDistanceArcsec: 0.025 in
# imposed by setting astrometry.maxMeanDistanceArcsec: 0.020 in
# the pipeline.
self.detector = 0
self.visit = 903344
Expand Down

0 comments on commit 4d3b23f

Please sign in to comment.