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

ImageDifferenceTask no longer timed in AP

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_pipe, ap_verify, verify
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP F21-3 (August)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      Timing information for Gen 3 image differencing stopped appearing in SQuaSH some time between August 1 and August 3. This bug appears in both the HiTS and COSMOS datasets, but does not appear in Gen 2. Investigate and find why the timing information is missing.

        Attachments

          Activity

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

          Just in case this is relevant, do the timers come out as log messages? We recently changed all gen3 output to python logging rather than lsst.log output.

          Show
          tjenness Tim Jenness added a comment - Just in case this is relevant, do the timers come out as log messages? We recently changed all gen3 output to python logging rather than lsst.log output.
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment -

          Tim Jenness, would you be willing to review this? It's 30 lines total, most of which are logging cleanup rather than the fix itself.

          Show
          krzys Krzysztof Findeisen added a comment - Tim Jenness , would you be willing to review this? It's 30 lines total, most of which are logging cleanup rather than the fix itself.
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Thanks for the log fixes. I haven't really encountered the target field in pipeline yaml before but this looks consistent with the other entries that are present.

          Show
          tjenness Tim Jenness added a comment - Thanks for the log fixes. I haven't really encountered the target field in pipeline yaml before but this looks consistent with the other entries that are present.
          Hide
          Permalink
          krzys Krzysztof Findeisen added a comment - - edited

          Thanks for the quick review. The target config field is specific to lsst.verify.tasks.commonMetrics.TimeMethodMetricTask and its subclasses.

          Show
          krzys Krzysztof Findeisen added a comment - - edited Thanks for the quick review. The target config field is specific to lsst.verify.tasks.commonMetrics.TimeMethodMetricTask and its subclasses.

            People

            Assignee:
            krzys Krzysztof Findeisen
            Reporter:
            krzys Krzysztof Findeisen
            Reviewers:
            Tim Jenness
            Watchers:
            Eric Bellm, Krzysztof Findeisen, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.