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

Move ap_verify_queries function into Ppdb

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Epic Link:
    • Sprint:
      AP S19-6, AP F19-1
    • Team:
      Alert Production

      Description

      DM-15588 removed the placeholder database APIs from ap_association, pipe, and verify. In doing this, a convenience function was created to compute a summary metric on the PPDB for use in ap_verify. This ticket will move that function and it's unittest into the Ppdb class and modifies ap_verify's usage of the function.

        Attachments

          Issue Links

          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-15588 Remove home-brewed SQLite PPDB

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            Agreed with Chris Morrison [X] and Eric Bellm on 2019-07-23 that this is not a current priority; dropping from the sprint.

            Show
            swinbank John Swinbank added a comment - Agreed with Chris Morrison [X] and Eric Bellm on 2019-07-23 that this is not a current priority; dropping from the sprint.
            Hide
            Permalink
            cmorrison Chris Morrison [X] (Inactive) added a comment -

            Worked on this as Andy Salnikov is reworking Ppdb and would like to see these queries in the code so he can incorporate in the future version.

            Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30264/pipeline

            Show
            cmorrison Chris Morrison [X] (Inactive) added a comment - Worked on this as Andy Salnikov is reworking Ppdb and would like to see these queries in the code so he can incorporate in the future version. Jenkins: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/30264/pipeline
            Hide
            Permalink
            cmorrison Chris Morrison [X] (Inactive) added a comment -

            For some reason only one pull request is showing up but they are all there.

            Show
            cmorrison Chris Morrison [X] (Inactive) added a comment - For some reason only one pull request is showing up but they are all there.
            Hide
            Permalink
            salnikov Andy Salnikov added a comment -

            Approved all three PRs, looks like a straightforward change. For non-SQL backend some verification-related queries may be less trivial to implement due to very different data organization. Once I have something ready we'll probably need to talk about that to see what could be done about that.

            Show
            salnikov Andy Salnikov added a comment - Approved all three PRs, looks like a straightforward change. For non-SQL backend some verification-related queries may be less trivial to implement due to very different data organization. Once I have something ready we'll probably need to talk about that to see what could be done about that.
            Hide
            Permalink
            cmorrison Chris Morrison [X] (Inactive) added a comment -

            Cool, thanks Andy. I'm not 100% sure, but I think at least the "nUnassociated" could be run on the LastDiaObject table instead of the full one if that makes things easier (and if that table is sticking around). The isVisitProcessed query will also get easier once we have a ccdVisit table somewhere. Not sure what else we'll end up needing for verify but that will be mostly up to Eric Bellm.

            Show
            cmorrison Chris Morrison [X] (Inactive) added a comment - Cool, thanks Andy. I'm not 100% sure, but I think at least the "nUnassociated" could be run on the LastDiaObject table instead of the full one if that makes things easier (and if that table is sticking around). The isVisitProcessed query will also get easier once we have a ccdVisit table somewhere. Not sure what else we'll end up needing for verify but that will be mostly up to Eric Bellm .

              People

              Assignee:
              cmorrison Chris Morrison [X] (Inactive)
              Reporter:
              cmorrison Chris Morrison [X] (Inactive)
              Reviewers:
              Andy Salnikov
              Watchers:
              Andy Salnikov, Chris Morrison [X] (Inactive), John Swinbank, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.