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

Move tests/negative.py from meas_algorithms to meas_base

    Details

    • Story Points:
      2
    • Team:
      Data Release Production

      Description

      Porting code from HSC to LSST brought over a unit test into meas_algorithms for functionality that exists in meas_base in LSST. This is due to the refactoring of code into meas_base on the LSST some while ago. This unit test currently runs with code from meas_algorithms, which means it can not simply be moved, as meas_base comes before meas_algorithms in the build order. This work may involve rewriting the unit test to use different code, or evaluating if it is worth bringing that functionality to meas_base along with the test. The code in question is the detection task.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Nate Lust: Can you be more specific about exactly which test this issue refers to? The title says "unit test negative", but grepping for "negative" in the meas_algorithms tests is uninformative. Please clarify the description of this issue and link it back to the original ticket on which the port was performed.

            Show
            swinbank John Swinbank added a comment - Nate Lust : Can you be more specific about exactly which test this issue refers to? The title says "unit test negative", but grepping for "negative" in the meas_algorithms tests is uninformative. Please clarify the description of this issue and link it back to the original ticket on which the port was performed.
            Hide
            nlust Nate Lust added a comment -

            The work was done on DM-5282. The reason you don't see it yet in source, is that the ticket is still pending review, and has not been merged. The unit test is called negative.py

            Show
            nlust Nate Lust added a comment - The work was done on DM-5282 . The reason you don't see it yet in source, is that the ticket is still pending review, and has not been merged. The unit test is called negative.py
            Hide
            jbosch Jim Bosch added a comment -

            As per comments, work has been completed in another (already linked) issue.

            Show
            jbosch Jim Bosch added a comment - As per comments, work has been completed in another (already linked) issue.

              People

              • Assignee:
                Unassigned
                Reporter:
                nlust Nate Lust
                Watchers:
                Jim Bosch, John Swinbank, Nate Lust
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: