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

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-16253 Switch obs_decam and obs_cfht to astro_metadata_translator

          • Done
          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15646 Rewrite header translation infrastructure for butler Gen 3

          • Done
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-539 Adopt astro_metadata_translator and decide on translator locations

          • Implemented
          relates to

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-539 Adopt astro_metadata_translator and decide on translator locations

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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:

                  Jenkins

                  No builds found.