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

Create a 'difference' functor method

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      2
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The compare* scripts from pipe_analysis compute the differences of various quantities between different data repositories.  To prepare for a future transition of pipe_analysis to using lsst.pipe.base.functors, it may be useful for a Functor to have a .difference method that allows you to pass two ParquetTable reference objects and have it return the difference in the calculations between the two.  

        Attachments

          Activity

          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Hi Chris- Can you review this quick ticket?  I just added a difference method to functors to allow simple calculation of differences between two different datasets, such as is done in the compare* scripts in pipe_analysis.

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Hi Chris- Can you review this quick ticket?  I just added a difference method to functors to allow simple calculation of differences between two different datasets, such as is done in the compare* scripts in pipe_analysis.
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          Looks good. Do you have a link to the Jenkins run you can attach to this ticket?

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - Looks good. Do you have a link to the Jenkins run you can attach to this ticket?
          Hide
          tmorton Tim Morton [X] (Inactive) added a comment -

          Thanks-- It's been awhile since I merged anything to master; do we need to run something Jenkins in addition to the automatic Travis check here? https://travis-ci.org/github/lsst/pipe_tasks/builds/731685656

          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Thanks-- It's been awhile since I merged anything to master; do we need to run something Jenkins in addition to the automatic Travis check here?  https://travis-ci.org/github/lsst/pipe_tasks/builds/731685656
          Hide
          cmorrison Chris Morrison [X] (Inactive) added a comment -

          That's just a flake8 test. Run a Jenkins job with your ticket branch here: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/activity/

          https://developer.lsst.io/work/flow.html

          Show
          cmorrison Chris Morrison [X] (Inactive) added a comment - That's just a flake8 test. Run a Jenkins job with your ticket branch here: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/activity/ https://developer.lsst.io/work/flow.html
          Show
          tmorton Tim Morton [X] (Inactive) added a comment - Jenkins run just started:  https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/32748/pipeline

            People

            Assignee:
            tmorton Tim Morton [X] (Inactive)
            Reporter:
            tmorton Tim Morton [X] (Inactive)
            Reviewers:
            Chris Morrison [X] (Inactive)
            Watchers:
            Chris Morrison [X] (Inactive), Lauren MacArthur, Tim Morton [X] (Inactive), Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.