Skip to content

Commit

Permalink
Remove unnecessary "require" overrides
Browse files Browse the repository at this point in the history
astrometry and photometry were already required in CalibrateTask
  • Loading branch information
parejkoj committed Sep 19, 2023
1 parent 0beac1b commit aa768b4
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 12 deletions.
4 changes: 0 additions & 4 deletions config/DECam/calibrate.py
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,6 @@
matchConfig.matcher.maxMatchDistArcSec = 2.0
matchConfig.sourceSelector.active.excludePixelFlags = False

# Demand astrometry and photoCal succeed
config.requireAstrometry = True
config.requirePhotoCal = True

config.doWriteMatchesDenormalized = True

config.measurement.load(os.path.join(obsConfigDir, "apertures.py"))
Expand Down
4 changes: 0 additions & 4 deletions config/HSC/calibrate.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,10 +48,6 @@
config.photoCal.match.referenceSelection.magLimit.maximum = 22.0
config.photoCal.colorterms.load(os.path.join(ObsConfigDir, 'colorterms.py'))

# Demand astrometry and photoCal succeed
config.requireAstrometry = True
config.requirePhotoCal = True

config.doWriteMatchesDenormalized = True

config.measurement.load(os.path.join(ObsConfigDir, "apertures.py"))
Expand Down
4 changes: 0 additions & 4 deletions config/LSSTCam-imSim/calibrate.py
Original file line number Diff line number Diff line change
Expand Up @@ -55,10 +55,6 @@
# select only stars for photometry calibration
config.photoCal.match.sourceSelection.unresolved.maximum = 0.5

# Demand astrometry and photoCal succeed
config.requireAstrometry = True
config.requirePhotoCal = True

# Activate calibration of measurements: required for aperture corrections
config.measurement.load(os.path.join(obsConfigDir, "apertures.py"))

Expand Down

0 comments on commit aa768b4

Please sign in to comment.