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

Implement MetricRegistry

    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

            Activity

            Hide
            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
            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:

                  Summary Panel