Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw, meas_astrom
-
Labels:None
-
Story Points:2
-
Epic Link:
-
Sprint:DRP F18-5
-
Team:Data Release Production
Description
There are two functions in the stack that effectively generate a denormalized catalog of matches. The matches between a reference and source catalog are "joined' into a single "match" catalog, putting a ref_ or src_ prefix on the field names for the reference and source fields, respectively. However, when the schema for the match catalog is created, it does not propagate any aliases that may be set, which will then break any code looking for a field by its alias name (often happens with aliases set for backwards compatibility). This ticket is to update the following functions to include the aliases from the individual catalogs in the schema for the matched catalog:
I will leave it for a future tickets whether these two very similar functions should be consolidated into one.
Attachments
Activity
Might you have a moment to give this a look on behalf of Jim and myself? You may note that there is no formal test for the function in afw. I believe I may be the only consumer at the moment (I use it in the pipe_analysis scripts, and indeed it was in running these that revealed the issue) and can confirm it now runs successfully with both old (VERSION < 3) and new catalogs, whereas, prior to this fix the backwards compatibility was broken. As I note in the description, extra effort would be better placed in seeing if these very similar functions can be merged into one.
Looks good. I had one minor comment on wording in afw.
Thanks for the speedy review. I addressed your comment, rebased (some churn in afw – including a revert today), re-Jenkinsed (passed), and merged.
Pair coding with Jim Bosch