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

Remove aliased afwGeom geom usage from stack

    XMLWordPrintable

    Details

    • Story Points:
      6
    • Sprint:
      Arch 2019-07-15, Arch 2019-07-22
    • Team:
      Architecture

      Description

      We have deprecated lsst.afw.geom.Point2I and related routines but they are still used all over the place. They need to be removed so that to allow DM-20565 to be unblocked.

        Attachments

          Issue Links

            Activity

            Hide
            sullivan Ian Sullivan added a comment -

            I reviewed: meas_modelfit, ip_diffim, meas_deblender, and pipe_tasks.

            I can look at the remaining packages if needed.

            Show
            sullivan Ian Sullivan added a comment - I reviewed: meas_modelfit, ip_diffim, meas_deblender, and pipe_tasks. I can look at the remaining packages if needed.
            Hide
            sullivan Ian Sullivan added a comment -

            I believe I have finished reviewing all of the packages. Please let me know if I've missed any.

            Show
            sullivan Ian Sullivan added a comment - I believe I have finished reviewing all of the packages. Please let me know if I've missed any.
            Hide
            tjenness Tim Jenness added a comment -

            Thank you very much Ian Sullivan. You have reviewed all the open pull requests. There are now only a handful of packages remaining (including jointcal and obs_subaru).

            Show
            tjenness Tim Jenness added a comment - Thank you very much Ian Sullivan . You have reviewed all the open pull requests. There are now only a handful of packages remaining (including jointcal and obs_subaru).
            Hide
            tjenness Tim Jenness added a comment -

            Simon Krughoff can you please review obs_sdss and obs_lsstSim changes. Mostly trivial although I fixed a couple of ResourceWarnings in obs_sdss.

            Show
            tjenness Tim Jenness added a comment - Simon Krughoff can you please review obs_sdss and obs_lsstSim changes. Mostly trivial although I fixed a couple of ResourceWarnings in obs_sdss.
            Hide
            tjenness Tim Jenness added a comment -

            I'm going to mark this ticket as complete. It's about 98% complete. meas_mosaic has not been touched but that package is imminently disappearing. The remainder are a few example files and the odd bit of support code that don't stop tests failing. We will need to do a new check when we remove the afwGeom wrappers since we currently aren't preventing people adding new code with afwGeom usage in it.

            Show
            tjenness Tim Jenness added a comment - I'm going to mark this ticket as complete. It's about 98% complete. meas_mosaic has not been touched but that package is imminently disappearing. The remainder are a few example files and the odd bit of support code that don't stop tests failing. We will need to do a new check when we remove the afwGeom wrappers since we currently aren't preventing people adding new code with afwGeom usage in it.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Ian Sullivan, John Parejko
              Watchers:
              Ian Sullivan, John Parejko, John Swinbank, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.