Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: obs_base
-
Labels:None
-
Story Points:1
-
Epic Link:
-
Sprint:DRP F16-6
-
Team:Data Release Production
Description
The butler.get(dataset + "_md", dataId) pattern is only supported for images, not catalogs. It would be useful to allow this to work on catalogs.
At the same time, it would be useful to add support for getting the length and schema of a catalog.
Attachments
Issue Links
Activity
First Jenkins revealed a small fix was needed. A new Jenkins run passed.
Paul Price
added a comment - First Jenkins revealed a small fix was needed. A new Jenkins run passed. Thanks for cleaning this up. See comments on github.
Made changes that were requested. A new Jenkins is running.
Paul Price
added a comment - Made changes that were requested. A new Jenkins is running. Jenkins passed following a rebase.
Paul Price
added a comment - Jenkins passed following a rebase. Sorry, that Jenkins was for another issue (not even mine).
The relevant Jenkins is this one.
Paul Price
added a comment - Sorry, that Jenkins was for another issue (not even mine).
The relevant Jenkins is this one . Merged to master.
Paul Price
added a comment - Merged to master. 
John Parejko, would you mind reviewing this, please? There's some cleanup as well as adding the new functionality. It's currently going through Jenkins.
price@price-laptop:~/LSST/obs/base (tickets/DM-8212=) $ git subcommit 063ae18cb95808ac13edce0dc05a0ce15c91f9b8Author: Paul Price <price@astro.princeton.edu>Date: Tue Nov 8 15:04:06 2016 -0500CameraMapper: simplify setting of extra dataset featuresExtra dataset features (e.g., "raw_sub", "raw_md", "raw_filename") areare created from the original mapping (e.g., "raw") by defining a bunchof extra methods. This patch replaces the sea ofif not hasattr(self, someName):setattr(self, someName, someImpl)calls with a common function to set the implementations for each extradataset feature. This clarifies the code and makes it easier to add moreextra features.python/lsst/obs/base/cameraMapper.py | 97 +++++++++++++++++-------------------1 file changed, 47 insertions(+), 50 deletions(-)commit cac031cc03abf537bf3c75879da0c40a65993666Author: Paul Price <price@astro.princeton.edu>Date: Tue Nov 8 15:07:19 2016 -0500CameraMapper: add extra dataset features for catalogsAdded "foo_md", "foo_len" and "foo_schema" if "foo" is a catalog.Note that the "foo_schema" commonly has a different meaning because it'susually defined explicitly if the "foo" is an important pipeline output.python/lsst/obs/base/cameraMapper.py | 15 +++++++++++++++tests/MinMapper2.paf | 10 +++++-----tests/testCameraMapper.py | 33 ++++++++++++++++++++++++++++++---3 files changed, 50 insertions(+), 8 deletions(-)