Details
-
Type:
Bug
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Team:Data Release Production
Description
In recent runs of the stack through multiBandDriver.py, the persisted forced src tables for the coadds are not getting ra and dec set properly (all entries for the coord_ra and coord_dec columns are NaN). Looking back at a run in mid-Nov, 2016, these numbers were indeed set properly in the forced tables. Assuming this was not intentional, track down the cause and fix it such that these values get set properly for the persisted forced src tables.
Attachments
Issue Links
- blocks
-
DM-9907 Release hscPipe 5.0-beta2
- Done
Activity
Also need:
price@pap-laptop:~/LSST/meas_modelfit (tickets/DM-9556=) $ git sub
|
commit f2c37e1e4543ce5b5ffe670d2c149d8c426ca6b9
|
Author: Paul Price <price@astro.princeton.edu>
|
Date: Fri Mar 31 13:35:24 2017 -0400
|
|
|
tests: adapt to ForcedMeasurementConfig changes
|
|
ForcedMeasurementConfig now by default copies fields that aren't
|
present in our basic test catalogs, so set the list of columns
|
to copy to what we do have (the same setting as was default previously).
|
|
|
tests/testDoubleShapeletPsfApprox.py | 1 +
|
tests/testGeneralShapeletPsfApproxPlugins.py | 1 +
|
2 files changed, 2 insertions(+)
|
Plus similar changes in meas_extensions_photometryKron.
Paul Price
added a comment - Also need:
price@pap-laptop:~/LSST/meas_modelfit (tickets/DM-9556=) $ git sub
commit f2c37e1e4543ce5b5ffe670d2c149d8c426ca6b9
Author: Paul Price <price@astro.princeton.edu>
Date: Fri Mar 31 13:35:24 2017 -0400
tests: adapt to ForcedMeasurementConfig changes
ForcedMeasurementConfig now by default copies fields that aren't
present in our basic test catalogs, so set the list of columns
to copy to what we do have (the same setting as was default previously).
tests/testDoubleShapeletPsfApprox.py | 1 +
tests/testGeneralShapeletPsfApproxPlugins.py | 1 +
2 files changed, 2 insertions(+)
Plus similar changes in meas_extensions_photometryKron. Jim Bosch, there are four new changes to review, required in tests of forced measurement because we changed the defaults to require a field which isn't present without some extra effort. Changes are in meas_modelfit, meas_extensions_photometryKron, meas_extensions_convolved and pipe_tasks. They're pretty simple, but please have a quick look at them before I merge.
Paul Price
added a comment - Jim Bosch , there are four new changes to review, required in tests of forced measurement because we changed the defaults to require a field which isn't present without some extra effort. Changes are in meas_modelfit, meas_extensions_photometryKron, meas_extensions_convolved and pipe_tasks. They're pretty simple, but please have a quick look at them before I merge. Looks fine. Too bad it's so fragile, but I don't see a simple way to fix that, and it's certainly out of scope here.
Thanks Jim.
Merged to master.
Paul Price
added a comment - Thanks Jim.
Merged to master. 
Revised:
-price@pap-laptop:~/LSST/meas_base (tickets/DM-9556=) $ git sub-patchcommit 94f876db450f3494ff1ef8d737c696a4586f7836Author: Paul Price <price@astro.princeton.edu>Date: Fri Mar 31 11:48:21 2017 -0400forcedMeasurement: copy coordinates, deblend_nChildForced measurement catalogs currently have no RA,Dec ordeblend_nChild. These used to be copied from the referencecatalog, but commit 448121a4 removed this behaviour.Restoring it.diff --git a/python/lsst/meas/base/forcedMeasurement.py b/python/lsst/meas/base/forcedMeasurement.pyindex 52dcbb3..1c97858 100644--- a/python/lsst/meas/base/forcedMeasurement.py+++ b/python/lsst/meas/base/forcedMeasurement.py@@ -163,7 +163,8 @@ class ForcedMeasurementConfig(BaseMeasurementConfig):copyColumns = lsst.pex.config.DictField(keytype=str, itemtype=str, doc="Mapping of reference columns to source columns",- default={"id": "objectId", "parent": "parentObjectId"}+ default={"id": "objectId", "parent": "parentObjectId", "deblend_nChild": "deblend_nChild",+ "coord_ra": "coord_ra", "coord_dec": "coord_dec"})checkUnitsParseStrict = lsst.pex.config.Field(