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

Add FOCUSZ into ExposureInfo

    XMLWordPrintable

Details

    Description

      FOCUSZ is not currently in ExposureInfo. We want to add this in so we can access it with the images along with the unit information. As I understand it this requires two steps:

      1) Add the focus_z information into ObservationInfo in astro_metadata_translator.

      2) Then it can be added to ExposureInfo.

      Questions I have:

      Should it go into VisitInfo? 

      How can you add distance units to values in the C++? I see you can use the lsst.geom units for angular things but not sure about linear distance.

      Attachments

        Activity

          Requesting reviews on PRs in:

          afw: https://github.com/lsst/afw/pull/646

          obs_base: https://github.com/lsst/obs_base/pull/425 

          Also, please let me know if there's anywhere else I still need to make changes. Thanks!

          jbkalmbach Bryce Kalmbach added a comment - Requesting reviews on PRs in: afw: https://github.com/lsst/afw/pull/646 obs_base: https://github.com/lsst/obs_base/pull/425   Also, please let me know if there's anywhere else I still need to make changes. Thanks!

          jbkalmbach, are you ok with me taking over the afw portion of the review?

          krzys Krzysztof Findeisen added a comment - jbkalmbach , are you ok with me taking over the afw portion of the review?

          Yeah, of course. Thanks for taking it on!

          jbkalmbach Bryce Kalmbach added a comment - Yeah, of course. Thanks for taking it on!

          No objections to the afw part.

          krzys Krzysztof Findeisen added a comment - No objections to the afw part.
          tjenness Tim Jenness added a comment -

          obs_base changes seem fine.

          tjenness Tim Jenness added a comment - obs_base changes seem fine.

          People

            jbkalmbach Bryce Kalmbach
            jbkalmbach Bryce Kalmbach
            Tim Jenness
            Bryce Kalmbach, Krzysztof Findeisen, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.