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

Reimplement FitsStorage support for direct PropertyList reads

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_persistence
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      DRP F18-4
    • Team:
      Data Release Production

      Description

      The move of the implementation of FitsStorage on DM-15599 neglected to include support for reading PropertyLists directly from FITS.  This is somewhat understandable; we had no test coverage for that functionality and it was used in only one place (obs_lsstCam) that's not yet in CI.  Should be easy to add it back in.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            Ready for review; implementation is in daf_persistence, tests are in obs_test (can't go lower than that because functionality needs a real-ish Butler and test repo).

            Jenkins run is underway.

            Show
            jbosch Jim Bosch added a comment - Ready for review; implementation is in daf_persistence, tests are in obs_test (can't go lower than that because functionality needs a real-ish Butler and test repo). Jenkins run is underway.
            Hide
            rhl Robert Lupton added a comment -

            One comment on GitHub, but good to go

            Show
            rhl Robert Lupton added a comment - One comment on GitHub, but good to go

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Robert Lupton
                Watchers:
                Jim Bosch, Robert Lupton
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel