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

Let MetricsControllerTask store application-specific metadata

    XMLWordPrintable

Details

    • Improvement
    • Status: Done
    • Resolution: Done
    • None
    • verify
    • None

    Description

      As originally designed, lsst.verify.compatibility.MetricsControllerTask can store only metadata that is required (e.g., by SQuaSH) for all metrics. However, applications like ap_verify, validate_drp, etc. may wish to store information that's specific to their workflow.

      Design and implement a system for passing generic Job-level metadata to MetricsControllerTask.

      Attachments

        Issue Links

          Activity

            Proposed API: add an optional argument to MetricsControllerTask.runDataRef that takes a dictionary of metadata keys and values. This is more work for the client than registering metadata once with the class, but it's more flexible. Given that the SQuaSH-required metadata (specifically, the data ID) is already different for different runs, it's reasonable to support run-specific values for application-specific metadata as well.

            krzys Krzysztof Findeisen added a comment - Proposed API: add an optional argument to MetricsControllerTask.runDataRef that takes a dictionary of metadata keys and values. This is more work for the client than registering metadata once with the class, but it's more flexible. Given that the SQuaSH-required metadata (specifically, the data ID) is already different for different runs, it's reasonable to support run-specific values for application-specific metadata as well.

            Hi ebellm, can you take a look at this 20-line change? Thanks!

            krzys Krzysztof Findeisen added a comment - Hi ebellm , can you take a look at this 20-line change? Thanks!
            ebellm Eric Bellm added a comment -

            Looks good!

            ebellm Eric Bellm added a comment - Looks good!

            People

              krzys Krzysztof Findeisen
              krzys Krzysztof Findeisen
              Eric Bellm
              Eric Bellm, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.