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

Enable lsst.verify to subset specifications by a complete metadata match

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Verification
    • Labels:

      Description

      You can currently subset (query for) specifications by passing a metadata term dictionary to the Jobs.specs.subset() method. This kind of query was intended to be permissive and just match any specification that could apply given a Job's metadata. This meant that if the Job metadata had an extra term that the Specification does not filter by, that specification would still be applicable to the job.

      For DM-12253, Simon Krughoff needs to subset specifications by an exact match to the input metadata query terms. This means that if the queried metadata has terms not present in a Specification's metadata, that specification does not match.

      This ticket is to implement that alternative query pattern.

        Attachments

          Issue Links

            Activity

            Hide
            jsick Jonathan Sick added a comment -

            See the PR for an example. This should let you find specifications for validate_drp now.

            Show
            jsick Jonathan Sick added a comment - See the PR for an example. This should let you find specifications for validate_drp now.
            Hide
            krughoff Simon Krughoff added a comment -

            Looks fine. Mostly minor comments. The only one of substance is that I don't understand how the new test passes.

            Show
            krughoff Simon Krughoff added a comment - Looks fine. Mostly minor comments. The only one of substance is that I don't understand how the new test passes.

              People

              • Assignee:
                jsick Jonathan Sick
                Reporter:
                jsick Jonathan Sick
                Reviewers:
                Simon Krughoff
                Watchers:
                Jonathan Sick, Simon Krughoff
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel