Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-15914

Use astro_metadata_translator in daf_butler

    XMLWordPrintable

    Details

      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.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Jim Bosch I hope you can review these changes. All tests pass in obs_base and obs_subaru. To test it you'll need branch tickets/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.

            Show
            tjenness Tim Jenness added a comment - Jim Bosch I hope you can review these changes. All tests pass in obs_base and obs_subaru. To test it you'll need branch tickets/ 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.
            Hide
            jbosch Jim Bosch added a comment -

            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.

            Show
            jbosch Jim Bosch added a comment - 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.
            Hide
            tjenness Tim Jenness added a comment -

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

            Show
            tjenness Tim Jenness added a comment - All branches merged. We are now using astro_metadata_translator for obs_subaru.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Jim Bosch
              Watchers:
              Jim Bosch, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: