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

Implement PropertySet.getitem and return get()

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_base
    • Labels:
      None

      Description

      Following the adoption of RFC-596:

      • Add getitem support (returning scalars in all cases)
      • Undeprecate get(), modifying it to return scalars and to support a default value.
      • Investigate adding update() method.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Russell Owen the PR implements RFC-596: getitem and get now call getScalar. Also added update() method to match dict (although I haven't added support for kwargs). This PR requires that DM-19877 is merged first.

            Show
            tjenness Tim Jenness added a comment - Russell Owen the PR implements RFC-596 : getitem and get now call getScalar. Also added update() method to match dict (although I haven't added support for kwargs). This PR requires that DM-19877 is merged first.
            Hide
            rowen Russell Owen added a comment -

            Overall looks great. A few comments on github

            Show
            rowen Russell Owen added a comment - Overall looks great. A few comments on github
            Hide
            tjenness Tim Jenness added a comment -

            Thanks for your review. Merged.

            Show
            tjenness Tim Jenness added a comment - Thanks for your review. Merged.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Russell Owen
                Watchers:
                Kian-Tat Lim, Russell Owen, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel