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

Add items() method to PropertySet/PropertyList

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_base
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      Arch 2019-09-16
    • Team:
      Architecture

      Description

      Prior to DM-16297 when a header was merged in astro_metadata_translator the resultant header was always a dict and never a PropertyList. In DM-16297 that is changing because PropertyX can now support for dict-like methods and the conversion to a dict is not required, and also the result of the merge must be able to be sent to afw and it is important that the type does not change.

      This breaks pipe_drivers which expects the result from merge_headers to support the items() method and PropertyX does not. On this ticket I will add items() using the existing _iter_ support.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            This is a very small change that copies items and values from collections.abc to compensate for PropertySet not being able to inherit from collections.abc.

            Show
            tjenness Tim Jenness added a comment - This is a very small change that copies items and values from collections.abc to compensate for PropertySet not being able to inherit from collections.abc.

              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:

                  Summary Panel