This is a fairly simple change to convert AFW unit tests to work with version 1 tables. Version 1 table use "_" to separate the components of field names. They also use all scalar type instead of compound types, like Point , Quadrupole ant Matrix.
Note that some of the unit tests actually test version 0 functionality, and have not been changed.
Lauren, you can pass this to John if you don't have afw table background, but I think you probably have the right experience.
--------------------------------------------------------
afw and meas_base were modified.
pgeepc2:/sandbox2/pgee/mylsst9/Linux64/afw> git diff --stat master
src/table/Schema.cc | 20 +++++++---
tests/sourceMatch.py | 7 ++--
tests/testFunctorKeys.py | 20 ++++------
tests/testSchema.py | 33 ++++++++--------
tests/testSourceTable.py | 97 +++++++++++++++++++++++++---------------------
tests/ticket2707.py | 1 -
6 files changed, 94 insertions
, 84 deletions
include/lsst/meas/base/CentroidUtilities.h | 28 ++++++++++++++++++++++
include/lsst/meas/base/ShapeUtilities.h | 36 ++++++++++++++++++++++++++++
src/CentroidUtilities.cc | 6 +++++
src/ShapeUtilities.cc | 9 +++++++
4 files changed, 79 insertions
I would like clarification on the scope of this change. Note that the routine schema.getNames() is still coded assuming that the common separator in field names is the ".". Is it our intention to replace this with "_", or is this routine only intended for use with version 0 tables?