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

Additional Filter deprecation warnings in lsstCamMapper

    XMLWordPrintable

    Details

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

      Description

      There are a few unwrapped uses of afwImage.Filter in lsstCamMapper.py that throw deprecation warnings. These should at least be suppressed.

      There might be other uses in the stack that come up with searching for Filter.getNames.

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-27170 Deprecate and phase out afw::image::Filter

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-28577 Surpress Filter warnings in FilterDefinitions

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            erykoff Eli Rykoff created issue -
            Parejkoj John Parejko made changes -
            Field Original Value New Value
            Link This issue is triggered by DM-27170 [ DM-27170 ]
            Parejkoj John Parejko made changes -
            Link This issue relates to DM-28577 [ DM-28577 ]
            Parejkoj John Parejko made changes -
            Epic Link DM-27911 [ 442604 ]
            Sprint AP S21-3 (February) [ 1072 ]
            Story Points 0.5
            Team Alert Production [ 10300 ]
            Parejkoj John Parejko made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            I think this is the only one that needs it, the one in the coadd exposure id code is actually DM-28166, and not a "we know this is deprecated just shut it up" situation, I think. Does this cleanup the warnings you were seeing?

            Show
            Parejkoj John Parejko added a comment - I think this is the only one that needs it, the one in the coadd exposure id code is actually DM-28166 , and not a "we know this is deprecated just shut it up" situation, I think. Does this cleanup the warnings you were seeing?
            Parejkoj John Parejko made changes -
            Reviewers Eli Rykoff [ erykoff ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            erykoff Eli Rykoff added a comment -

            Looks good, sorry for the delay. Seems to need a rebase at this point.

            Show
            erykoff Eli Rykoff added a comment - Looks good, sorry for the delay. Seems to need a rebase at this point.
            erykoff Eli Rykoff made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33678/pipeline

            Show
            Parejkoj John Parejko added a comment - Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/33678/pipeline
            Parejkoj John Parejko made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            sullivan Ian Sullivan made changes -
            Rank Ranked higher

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              erykoff Eli Rykoff
              Reviewers:
              Eli Rykoff
              Watchers:
              Eli Rykoff, John Parejko
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.