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

Make cameraGeom::Camera round-trip persistable

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
    • Story Points:
      6
    • Epic Link:
    • Sprint:
      BG3_F18_10
    • Team:
      Data Release Production

      Description

      Initial plan is to do this with afw::table::io, as I think it'll be easiest and I think we'll need it eventually even if we implement some other format first (so we can put Detectors with back-links into Exposures).  YAML and pickle are the other options.

      This is needed for Gen3 obs integration.

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-14363 Make afw::cameraGeom::Detector table-persistable

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-14980 Reinstate the ability of a Detector to find the Camera in which it lives

          • Done
          is duplicated by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15918 Make Camera round-trip persistable via afw::table::io

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Simon Krughoff, mind taking this review?  It's may be a 1400-line change, it's all pretty straightforward: it just moves Camera and DetectorCollection to C++ (with pybind11 wrappers), then makes them persistable.

            If you don't think you'll be able to get to it by, say, middle of next week, please let me know, and I'll go looking for another victim.

            Show
            jbosch Jim Bosch added a comment - Simon Krughoff , mind taking this review?  It's may be a 1400-line change, it's all pretty straightforward: it just moves Camera and DetectorCollection to C++ (with pybind11 wrappers), then makes them persistable. If you don't think you'll be able to get to it by, say, middle of next week, please let me know, and I'll go looking for another victim.
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            GitHub PR for afw (the only affected package) is https://github.com/lsst/afw/pull/403.  Note that this is a diff against DM-14363, which is a blocker being reviewed simultaneously.

            Show
            jbosch Jim Bosch added a comment - GitHub PR for afw (the only affected package) is https://github.com/lsst/afw/pull/403.   Note that this is a diff against DM-14363 , which is a blocker being reviewed simultaneously.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            I should be able to get to it.

            Show
            krughoff Simon Krughoff added a comment - I should be able to get to it.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            This looks good to me.

            Show
            krughoff Simon Krughoff added a comment - This looks good to me.

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Simon Krughoff
              Watchers:
              Jim Bosch, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.