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

Implement MetricRegistry

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: verify
    • Labels:
      None
    • Story Points:
      4
    • Epic Link:
    • Sprint:
      AP S19-1, AP S19-2
    • Team:
      Alert Production

      Description

      DM-16017 implemented only a very minimal version of MetricsControllerTask, which had the metrics it runs, and their configurations, hard-coded. The class needs to be generalized to take a configurable set of metrics.

      This ticket should implement MetricRegistry and @register as described in DMTN-098, including demonstrating that a config file to MetricsControllerTask can both create and independently configure multiple MetricTask instances.

        Attachments

          Issue Links

          blocks

          Improvement - An improvement or enhancement to an existing feature or task. DM-16536 Migrate all metrics from ap.verify.measurements

          • Done
          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-16017 Prototype a metrics-handling Task

          • Done
          relates to

          Bug - A problem which impairs or prevents the functions of the product. DM-17283 Must specify low-level modules when using :lsst-config-field:

          • To Do

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            krzys Krzysztof Findeisen added a comment -

            Hi John Parejko, would you be willing to review this ticket? It's about 700 lines, mostly in lsst.verify.

            Show
            krzys Krzysztof Findeisen added a comment - Hi John Parejko , would you be willing to review this ticket? It's about 700 lines, mostly in lsst.verify .
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Sorry it took me so long to get to this! You should have pestered me sooner.

            See the comments on the PR. Nothing huge, but there are some things to be clarified. I can almost see how to use this for my own Metrics, but not quite.

            Show
            Parejkoj John Parejko added a comment - Sorry it took me so long to get to this! You should have pestered me sooner. See the comments on the PR. Nothing huge, but there are some things to be clarified. I can almost see how to use this for my own Metrics, but not quite.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              John Parejko
              Watchers:
              John Parejko, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.