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

Remove unnecessary connection from DetectAndMeasureTask

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ip_diffim
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      AP F22-3 (August)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      DetectAndMeasureTask includes an input connection for selectSources, which is not actually used. This connection and the corresponding documentation should be removed.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-35721 Create mocks of the new image differencing for ap_verify

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            sullivan Ian Sullivan added a comment -

            PRs:
            https://github.com/lsst/ip_diffim/pull/232
            https://github.com/lsst/ap_verify/pull/168

            Show
            sullivan Ian Sullivan added a comment - PRs: https://github.com/lsst/ip_diffim/pull/232 https://github.com/lsst/ap_verify/pull/168
            Hide
            Permalink
            sullivan Ian Sullivan added a comment -

            Passes on Jenkins including ci_hsc and ci_imsim: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/37232/

            Show
            sullivan Ian Sullivan added a comment - Passes on Jenkins including ci_hsc and ci_imsim: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/37232/
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Looks fine. Are there any other API changes we need to make before people start using this more broadly?

            Show
            Parejkoj John Parejko added a comment - Looks fine. Are there any other API changes we need to make before people start using this more broadly?

              People

              Assignee:
              sullivan Ian Sullivan
              Reporter:
              sullivan Ian Sullivan
              Reviewers:
              John Parejko
              Watchers:
              Ian Sullivan, John Parejko
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.