Paul Price added a comment - 15/Sep/17 11:28 AM - edited pprice@lsst-dev01:~/LSST/obs/subaru[tickets/DM-11927] $ git sub commit 8fd8481a642ad8209e7b1c842ff54edba9fd6d1a Author: Paul Price <price@astro.princeton.edu> Date: Fri Sep 15 13:26:09 2017 -0500   config: make matching consistent ProcessCcdTask uses multiple matchers; make their configuration consistent. Especially important is 'maxOffsetPix=250', which has been shown to be the sweet spot, and if some matchers don't have this set they can fail. Bug discovered and diagnosed by Hsin-Fang Chiang.   config/hsc/processCcd.py | 14 +++++++++----- 1 file changed, 9 insertions(+), 5 deletions(-)

Hsin-Fang Chiang added a comment - 15/Sep/17 12:11 PM Using w_2017_36 with this ticket change, I checked the 6 cases that ProcessCcd outcome changed in w_2017_36 ( DM-11189 ). For the 4 that failed in w_2017_36 but passed in w_2017_34 ( --id visit=24304 ccd=70 --id visit=28374 ccd=101 --id visit=28386 ccd=103 --id visit=9868 ccd=77 ), they now all passed. For the 2 that failed in w_2017_34 but passed in w_2017_36 ( --id visit=1206 ccd=77 --id visit=29324 ccd=99 ), they still pass with this ticket. So they are not identical with before DM-11356 (differenses in photoCal.matcher) but it's better. And I think it makes sense to have all matchers consistent too.

Paul Price added a comment - 15/Sep/17 12:14 PM Thanks Hsin-Fang Chiang ! Merged to master.