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

Port validate_drp to lsst.verify

    Details

      Description

      Port the validate_drp package to use the more current lsst.verify classes and persistence. This is corresponding with the updating of the jointcal package to use lsst.verify.

        Attachments

          Issue Links

            Activity

            Hide
            krughoff Simon Krughoff added a comment -

            Merged final commit in verify

            Show
            krughoff Simon Krughoff added a comment - Merged final commit in verify
            Hide
            jsick Jonathan Sick added a comment -

            Sweet.

            Show
            jsick Jonathan Sick added a comment - Sweet.
            Hide
            krughoff Simon Krughoff added a comment -

            I'm hoping you'd be willing to give this a quick look. I missed it when I was merging the other branches. It's a +6-1 single commit PR.

            Show
            krughoff Simon Krughoff added a comment - I'm hoping you'd be willing to give this a quick look. I missed it when I was merging the other branches. It's a +6-1 single commit PR.
            Hide
            krughoff Simon Krughoff added a comment -

            The code has been ported. I attach two versions of the report produced by reportPerformance.py. Some of the PA1 and PA2 numbers are slightly difference between the master version and the porting branch, but this is due to different random samples being picked between subsequent runs. This will be investigate more in DM-12570 to see if we should make measurements of those metrics deterministic.

            Note that the closing of this ticket does not indicate this has been merged to master. We are maintaining a longer running port branch, verify_port, to complete the porting including further testing between the port and master and for updating documentation.

            Show
            krughoff Simon Krughoff added a comment - The code has been ported. I attach two versions of the report produced by reportPerformance.py . Some of the PA1 and PA2 numbers are slightly difference between the master version and the porting branch, but this is due to different random samples being picked between subsequent runs. This will be investigate more in DM-12570 to see if we should make measurements of those metrics deterministic. Note that the closing of this ticket does not indicate this has been merged to master. We are maintaining a longer running port branch, verify_port , to complete the porting including further testing between the port and master and for updating documentation.
            Hide
            krughoff Simon Krughoff added a comment -

            report_performance_master.rst is the ReST report generated by the master stack as of the closing of this ticket
            report_performance_verify_port.rst is the ReST report generated by the porting branch verify_port that resulted from the ticket branch for this issue

            Show
            krughoff Simon Krughoff added a comment - report_performance_master.rst is the ReST report generated by the master stack as of the closing of this ticket report_performance_verify_port.rst is the ReST report generated by the porting branch verify_port that resulted from the ticket branch for this issue

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel