Use astro_metadata_translator in daf_butler

Description

Replace afw visitInfo with obs_metadata ObservationInfo inside daf_butler and obs_base. For the first test, includes updates of obs_subaru so that ci_hsc butler gen 3 ingest will use it.

Checklist

Lucidchart Diagrams

Issue Matrix

hide

Activity

Show:
Tim Jenness
October 22, 2018 at 4:23 PM

All branches merged. We are now using astro_metadata_translator for obs_subaru.

Jim Bosch
October 8, 2018 at 1:34 PM

Very nice!  Found a few print debug relics, and I had a couple of questions about why some code was in obs_base instead of obs_metadata.

Tim Jenness
October 5, 2018 at 2:56 PM

I hope you can review these changes. All tests pass in obs_base and obs_subaru. To test it you'll need branch tickets/https://rubinobs.atlassian.net/browse/DM-15646#icft=DM-15646 of obs_metadata (that's waiting on review). Hope you like the clean up this brings. At least all the bits are more clearly isolated now.

Done
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Labels

Reviewers

Jim Bosch

Story Points

RubinTeam

Sprint

None

Checklist

Created October 1, 2018 at 11:49 AM
Updated April 15, 2019 at 9:44 PM
Resolved October 22, 2018 at 4:23 PM