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
            No builds found.
            krzys Krzysztof Findeisen created issue -
            krzys Krzysztof Findeisen made changes -
            Field Original Value New Value
            Link This issue is blocked by DM-16017 [ DM-16017 ]
            swinbank John Swinbank made changes -
            Sprint AP S19-1 [ 825 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S19-1 [ 825 ]
            krzys Krzysztof Findeisen made changes -
            Rank Ranked higher
            krzys Krzysztof Findeisen made changes -
            Component/s verify [ 15038 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue blocks DM-16536 [ DM-16536 ]
            krzys Krzysztof Findeisen made changes -
            Sprint AP S19-1 [ 825 ]
            krzys Krzysztof Findeisen made changes -
            Rank Ranked lower
            krzys Krzysztof Findeisen made changes -
            Epic Link DM-16713 [ 235321 ]
            krzys Krzysztof Findeisen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            swinbank John Swinbank made changes -
            Sprint AP S19-1 [ 825 ] AP S19-1, AP S19-2 [ 825, 830 ]
            krzys Krzysztof Findeisen made changes -
            Link This issue relates to DM-17283 [ DM-17283 ]
            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 .
            krzys Krzysztof Findeisen made changes -
            Reviewers John Parejko [ parejkoj ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            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.
            Parejkoj John Parejko made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            krzys Krzysztof Findeisen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              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.