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

            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Link This issue relates to DM-15646 [ DM-15646 ]
            tjenness Tim Jenness made changes -
            Risk Score 0
            tjenness Tim Jenness made changes -
            Epic Link DM-14487 [ 83298 ]
            tjenness Tim Jenness made changes -
            Component/s obs_base [ 10719 ]
            Description Replace afw visitInfo with obs_metadata ObservationInfo inside daf_butler. Replace afw visitInfo with obs_metadata ObservationInfo inside daf_butler and obs_base
            tjenness Tim Jenness added a comment -

            jbosch 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.

            tjenness Tim Jenness added a comment - jbosch 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.
            tjenness Tim Jenness made changes -
            Reviewers Jim Bosch [ jbosch ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-15646 [ DM-15646 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-15646 [ DM-15646 ]
            tjenness Tim Jenness made changes -
            Sprint BG3_F18_10 [ 797 ]
            tjenness Tim Jenness made changes -
            Issue Type Bug [ 1 ] Story [ 10001 ]
            tjenness Tim Jenness made changes -
            Description Replace afw visitInfo with obs_metadata ObservationInfo inside daf_butler and obs_base 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.
            tjenness Tim Jenness made changes -
            Story Points 4 6
            tjenness Tim Jenness made changes -
            Component/s obs_subaru [ 10747 ]
            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.

            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.
            jbosch Jim Bosch made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to RFC-539 [ RFC-539 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggered by RFC-539 [ RFC-539 ]
            tjenness Tim Jenness made changes -
            Component/s astro_metadata_translator [ 15700 ]
            tjenness Tim Jenness made changes -
            Link This issue blocks DM-16253 [ DM-16253 ]
            tjenness Tim Jenness added a comment -

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

            tjenness Tim Jenness added a comment - All branches merged. We are now using astro_metadata_translator for obs_subaru.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            swinbank John Swinbank made changes -
            Component/s obs_metadata [ 15508 ]
            tjenness Tim Jenness made changes -
            Summary Use obs_metadata in daf_butler Use astro_metadata_translator in daf_butler

            People

              tjenness Tim Jenness
              tjenness Tim Jenness
              Jim Bosch
              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.