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

Remove pex logging from base package

    XMLWordPrintable

Details

    • Story
    • Status: Done
    • Resolution: Done
    • None
    • base
    • None
    • Architecture

    Description

      Base still use pex loggin in file:

      python/lsst/base/packages.py

       

      in line 162.

      Default logging shall be used instead and  "warning" shall be used instead of warn in line 163.

      Attachments

        Activity

          tjenness Tim Jenness added a comment -

          The main question is whether this should be warnings.warn or log.warning. ktl do you have an opinion?

          tjenness Tim Jenness added a comment - The main question is whether this should be warnings.warn or log.warning . ktl do you have an opinion?
          price Paul Price added a comment -

          I'm fine with this once tjenness 's question has been resolved and the solution has been tested.

          price Paul Price added a comment - I'm fine with this once tjenness 's question has been resolved and the solution has been tested.

          I think this should be a log.warning.  My feeling is that warnings.warn should be reserved for Python itself or very-commonly-used Python libraries.  Since it uses a totally different filtering mechanism from all other LSST messages, we should almost never use it.

          ktl Kian-Tat Lim added a comment - I think this should be a log.warning .  My feeling is that warnings.warn should be reserved for Python itself or very-commonly-used Python libraries.  Since it uses a totally different filtering mechanism from all other LSST messages, we should almost never use it.

          People

            gcomoretto Gabriele Comoretto [X] (Inactive)
            gcomoretto Gabriele Comoretto [X] (Inactive)
            Paul Price
            Gabriele Comoretto [X] (Inactive), Kian-Tat Lim, Paul Price, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.