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

Add support for creating TRACE loggers

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: utils
    • Labels:
    • Story Points:
      1
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      Currently people have to create TRACE loggers themselves with TRACEn+log.name. Provide a simple API for that in utils.logging.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Kian-Tat Lim can you please look at the change in utils to add getTraceLogger? It's a simple function that really just moves a line from another place to reuse it.

          Ian Sullivan can you please look at ip_diffim and meas_extensions_gaap? It seems that ip_diffim is the only real user of TRACE loggers in python and then only really in one file.

          Show
          tjenness Tim Jenness added a comment - Kian-Tat Lim can you please look at the change in utils to add getTraceLogger? It's a simple function that really just moves a line from another place to reuse it. Ian Sullivan can you please look at ip_diffim and meas_extensions_gaap? It seems that ip_diffim is the only real user of TRACE loggers in python and then only really in one file.
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          LGTM.

          Show
          ktl Kian-Tat Lim added a comment - LGTM.
          Hide
          Permalink
          sullivan Ian Sullivan added a comment -

          ip_diffim looks fine, but I didn't find a PR or branch in meas_extensions_gaap.

          Show
          sullivan Ian Sullivan added a comment - ip_diffim looks fine, but I didn't find a PR or branch in meas_extensions_gaap .
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Ian Sullivan sorry. I forgot to push the branch. Fixed now and ready for review (it's only a line).

          Show
          tjenness Tim Jenness added a comment - Ian Sullivan sorry. I forgot to push the branch. Fixed now and ready for review (it's only a line).

            People

            Assignee:
            tjenness Tim Jenness
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            Ian Sullivan
            Watchers:
            Ian Sullivan, Kian-Tat Lim, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.