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

Port validate_drp to lsst.verify

    XMLWordPrintable

    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

          is triggering

          Bug - A problem which impairs or prevents the functions of the product. DM-15208 validate_drp skipTEx is being ignored

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-12448 add additional branch to jenkins validate_drp job

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-12269 Enable lsst.verify to subset specifications by a complete metadata match

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-9534 Output jointcal metrics via a metrics logger

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            jsick Jonathan Sick added a comment -

            Sweet.

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

            Merged final commit in verify

            Show
            krughoff Simon Krughoff added a comment - Merged final commit in verify

              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: