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

Un-deprecate old APIs related to afw.math.Background

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      4
    • Sprint:
      AP S20-5 (April)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      Per RFC-687, remove the deprecated status from certain methods associated with Background, BackgroundList, and BackgroundControl. Specifically:

      • revert lsst/afw@7038c57, 488744f, and 29ec3d3, removing the use of deprecation attributes/decorators for the code in question
      • after reverting these changes, update the documentation to not describe the affected components as deprecated

      This will leave the code with both "old" and "new" APIs for doing certain things; because of the past deprecation period, Stack code will be slightly biased towards the "new" APIs. Addressing this balance is out of scope for this ticket.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Promoting to April sprint per #dm-science-pipelines post. Should be merged by May 1.

            Show
            krzys Krzysztof Findeisen added a comment - Promoting to April sprint per #dm-science-pipelines post . Should be merged by May 1.
            Hide
            tjenness Tim Jenness added a comment -

            Looks okay. I worry about the getPixel message that is now left there.

            Show
            tjenness Tim Jenness added a comment - Looks okay. I worry about the getPixel message that is now left there.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Resolved by leaving getPixel deprecated, as before this ticket.

            Show
            krzys Krzysztof Findeisen added a comment - Resolved by leaving getPixel deprecated, as before this ticket.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              Tim Jenness
              Watchers:
              Krzysztof Findeisen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Due:
                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.