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

ImageDifferenceTask no longer timed in AP

    XMLWordPrintable

Details

    • Bug
    • Status: Done
    • Resolution: Done
    • None
    • ap_pipe, ap_verify, verify
    • None
    • 1
    • AP F21-3 (August)
    • Alert Production
    • 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

          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.

          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.

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

          krzys Krzysztof Findeisen added a comment - tjenness , would you be willing to review this? It's 30 lines total, most of which are logging cleanup rather than the fix itself.
          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.

          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.
          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.

          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

            krzys Krzysztof Findeisen
            krzys Krzysztof Findeisen
            Tim Jenness
            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.