Skip to content

Commit

Permalink
Use larger magnitude rejection sigma for astrometry
Browse files Browse the repository at this point in the history
This test data is sensitive to astrometric magnitude outlier rejection; we
have lower noise (and only 4 sources!) than real images, so Aperture vs. Psf
flux show different scatter, thus the `zpSigma` in the magnitude rejection
code is unrealistically small.
  • Loading branch information
parejkoj committed Nov 7, 2023
1 parent 281251e commit 5b0fd80
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions tests/test_calibrateImage.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,12 @@ def setUp(self):
self.config.psf_measure_psf.psfDeterminer = 'pca'
# We don't have many test points, so can't match on complicated shapes.
self.config.astrometry.matcher.numPointsForShape = 3
# ApFlux has more noise than PsfFlux (the latter unrealistically small
# in this test data), so we need to do magnitude rejection at higher
# sigma, otherwise we can lose otherwise good sources.
# TODO DM-39203: Once we are using Compensated Gaussian Fluxes, we
# will use those fluxes here, and hopefully can remove this.
self.config.astrometry.magnitudeOutlierRejectionNSigma = 9.0

# Something about this test dataset prefers the older fluxRatio here.
self.config.star_catalog_calculation.plugins['base_ClassificationExtendedness'].fluxRatio = 0.925
Expand Down

0 comments on commit 5b0fd80

Please sign in to comment.