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

Remove errant deprecation warning in multiBand.py

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_tasks
    • Labels:
      None
    • Story Points:
      2
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      When generating HSC pipeline graphs, I noticed a large number of deprecation warnings appear relating to MeasureMergedCoaddSourcesConnections. For example, when constructing the pipeline graph for HSC/DRP-ci_hsc.yaml:

      $ pipetask build -p $DRP_PIPE_DIR/pipelines/HSC/DRP-ci_hsc.yaml --show config > /dev/null
      		 
      py.warnings WARNING: /sdf/group/rubin/sw/conda/envs/lsst-scipipe-4.1.0/share/eups/Linux64/pipe_tasks/g3008f20bd0+85efd929dc/python/lsst/pipe/tasks/multiBand.py:261: UserWarning: MeasureMergedCoaddSourcesConnections.defaultTemplates is deprecated and no longer used. Use MeasureMergedCoaddSourcesConfig.inputCatalog.
      		 
        warnings.warn("MeasureMergedCoaddSourcesConnections.defaultTemplates is deprecated and no longer used. "

      The warning is generated in pipe/tasks/multiBand.py#L261 (and isn't specific to HSC).

      From this Slack thread discussion, it seems as if this deprecation warning can now be safely removed.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          fred3m Fred Moolekamp added a comment -

          Hi Lee,
          Would you mind a quick review of this ticket?

          Thanks,
          -Fred

          Show
          fred3m Fred Moolekamp added a comment - Hi Lee, Would you mind a quick review of this ticket? Thanks, -Fred
          Hide
          Permalink
          lskelvin Lee Kelvin added a comment -

          Thanks Fred! Can confirm that running my example snippet in the description of this ticket works as expected. I don't think a Jenkins is needed for this, LGTM! Cheers!

          Show
          lskelvin Lee Kelvin added a comment - Thanks Fred! Can confirm that running my example snippet in the description of this ticket works as expected. I don't think a Jenkins is needed for this, LGTM! Cheers!
          Hide
          Permalink
          fred3m Fred Moolekamp added a comment -

          Thanks for the quick review Lee!

          Show
          fred3m Fred Moolekamp added a comment - Thanks for the quick review Lee!

            People

            Assignee:
            fred3m Fred Moolekamp
            Reporter:
            lskelvin Lee Kelvin
            Reviewers:
            Lee Kelvin
            Watchers:
            Fred Moolekamp, Lee Kelvin
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.