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

Remove cameraGeom interfaces deprecated in DM-18610

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP S20-6 (May)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      This includes (at least):

      • Amplifier.getHasRawInfo()

       

      This ticket should not be merged until after the 19.0 release.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-20564 Do release 20.0.0 of science pipelines

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-18610 Add fields, limited mutability, and trim/assembly-state tracking to cameraGeom

          • Done
          mentioned in

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            swinbank John Swinbank added a comment - - edited

            The only deprecation I can see in cameraGeom is (per ticket) Amplifier.getHasRawInfo() (here and here).

            Show
            swinbank John Swinbank added a comment - - edited The only deprecation I can see in cameraGeom is (per ticket) Amplifier.getHasRawInfo() ( here and here ).
            Hide
            Permalink
            swinbank John Swinbank added a comment - - edited

            Hi Krzysztof Findeisen — do you have a moment to review this, please?

            PRs in

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/31729/pipeline/ (I've had to rebase since that Jenkins run, and will re-run Jenkins before the final merge to master).

            Show
            swinbank John Swinbank added a comment - - edited Hi Krzysztof Findeisen — do you have a moment to review this, please? PRs in afw and ip_isr Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/31729/pipeline/ (I've had to rebase since that Jenkins run, and will re-run Jenkins before the final merge to master ).
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Looks good, assuming getHasRawInfo really is the only method affected.

            Show
            krzys Krzysztof Findeisen added a comment - Looks good, assuming getHasRawInfo really is the only method affected.
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Thank you!

            Show
            swinbank John Swinbank added a comment - Thank you!

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              Jim Bosch, John Swinbank, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: