Tickets/dm 14868 #9
Conversation
For ease of running.
|
|
||
| setPhotocalConfigFromEups(config.calibrate.photoCal) | ||
| from lsst.meas.astrom.matchPessimisticB import MatchPessimisticBTask | ||
| config.calibrate.astrometry.matcher.retarget(MatchPessimisticBTask) |
There was a problem hiding this comment.
Don't we still need to retarget the photometric matcher, or does it use the same one?
There was a problem hiding this comment.
obs_decam/config/processCcd.py already defaults to the HTM catalogs for both the astrometric and photometric calibration.
It's actually done this since 2017-07-20. I was previously explicitly overriding that in the validation_data_decam/decamConfig.py to use the astrometry.net files.
There was a problem hiding this comment.
These new lines are necessary because obs_decam doesn't explicitly set MatchPessimisticBTask. Once DM-14868 is merged, it's likely that MatchPessimisticBTask will become the processCcd config default and I'll remove the lines above. I'd like to eventually remove any config customizations in the CI runs.
reprocess.sh
Outdated
| @@ -0,0 +1,17 @@ | |||
| setup obs_decam -t w_2018_37 | |||
| # To be run from within validation_data_decam repo | |||
| # were we were at DM-14868 | |||
There was a problem hiding this comment.
"were we were" --> "where we were"
Update curated processCcd output with HTM-ref-cat processed version.
Add explicit
reprocess.shscript for reference.Switch
decamConfig.pyto use MatchPessimisticBTask