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

Switch from the deprecated log.getName() to log.name

    XMLWordPrintable

Details

    • 2
    • DRP S21b
    • Data Release Production
    • No

    Description

      We are currently getting log-bombed with lines along the following:

      /software/lsstsw/stack_20210520/stack/miniconda3-py38_4.9.2-0.6.0/Linux64/ip_diffim/22.0.1-13-g00a9d746+721f44a764/python/lsst/ip/diffim/mo\
      delPsfMatch.py:494: FutureWarning: Call to deprecated method getName. (Use Python Logger compatible 'name' attribute. Will be removed after\
       v23.) -- Deprecated since version v23.
       log.log("TRACE4." + self.log.getName(), log.DEBUG,
      

      There are a number of instances in ip_diffim that need fixing, and it may be worth doing a quick search for any other occurrences in the stack.

      Attachments

        Issue Links

          Activity

            A quick search on GitHub also reveals instances in meas_extensions_piff and ap_verify. I will fix those here as well.

            lauren Lauren MacArthur added a comment - A quick search on GitHub also reveals instances in meas_extensions_piff and ap_verify . I will fix those here as well.

            Would you mind giving this a look when you get a chance? Jenkins of lsst_distrib lsst_ci ci_hsc ci_imsim ap_verify is running.

            lauren Lauren MacArthur added a comment - Would you mind giving this a look when you get a chance? Jenkins of lsst_distrib lsst_ci ci_hsc ci_imsim ap_verify is running .
            tjenness Tim Jenness added a comment -

            Two look okay. I think the Piff one needs a different fix because now that we have python logging the two logger lines are no longer needed at all. I have put a suggested change on the PR.

            tjenness Tim Jenness added a comment - Two look okay. I think the Piff one needs a different fix because now that we have python logging the two logger lines are no longer needed at all. I have put a suggested change on the PR.

            Thanks for such detailed explanations on the PRs Tim!  I've used your suggested changeset for meas_extensions_piff and the "identical reporting" option for ip_diffim.  I've kicked off another Jenkins and will wait for that to succeed before merging (so feel free to pipe in again in the next few hours if you see any remaining issues).

            lauren Lauren MacArthur added a comment - Thanks for such detailed explanations on the PRs Tim!  I've used your suggested changeset for meas_extensions_piff and the "identical reporting" option for ip_diffim .  I've kicked off another Jenkins and will wait for that to succeed before merging (so feel free to pipe in again in the next few hours if you see any remaining issues).
            tjenness Tim Jenness added a comment -

            It all looks great. Thanks.

            tjenness Tim Jenness added a comment - It all looks great. Thanks.

            Jenkins passed and (after a minor git hiccup) all three are merged.  Thanks again!

            lauren Lauren MacArthur added a comment - Jenkins passed and (after a minor git hiccup) all three are merged.  Thanks again!

            People

              lauren Lauren MacArthur
              lauren Lauren MacArthur
              Tim Jenness
              Lauren MacArthur, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.