Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-565

Retire coadd_chisquared and clean up coadd_utils

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      The coadd_chisquared package is moribund: it's not in regular use and has not seen any substantive development work in several years. It is included in lsst_apps and lsst_distrib by way of pipe_tasks' table file, but is not used by that package.

      The coadd_utils package provides some utility code (setCoaddEdgeBits, copyGoodPixels, CoaddDataIdContainer) which is used by the coaddition routines in pipe_tasks, and a Coadd base class which is only used by coadd_chisquared.

      I propose:

      • To drop the pipe_tasks dependency on coadd_chisquared;
      • To move the coadd_chisquared repository to https://github.com/lsst-dm/legacy-coadd_chisquared;
      • To remove the Coadd class and any supporting code from coadd_utils;
      • To remove the (deprecated and unused) makeBitMask function from coadd_utils.

        Attachments

          Issue Links

            Activity

            swinbank John Swinbank created issue -
            swinbank John Swinbank made changes -
            Field Original Value New Value
            Link This issue relates to DM-16078 [ DM-16078 ]
            swinbank John Swinbank made changes -
            Status Proposed [ 10805 ] Flagged [ 10606 ]
            swinbank John Swinbank made changes -
            Status Flagged [ 10606 ] Board Recommended [ 11405 ]
            swinbank John Swinbank made changes -
            Link This issue is triggering DM-17432 [ DM-17432 ]
            swinbank John Swinbank made changes -
            Link This issue is triggering DM-17433 [ DM-17433 ]
            swinbank John Swinbank made changes -
            Status Board Recommended [ 11405 ] Adopted [ 10806 ]
            gcomoretto Gabriele Comoretto [X] (Inactive) made changes -
            Remote Link This issue links to "Page (Confluence)" [ 19569 ]
            swinbank John Swinbank made changes -
            Resolution Done [ 10000 ]
            Status Adopted [ 10806 ] Implemented [ 11105 ]

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Watchers:
              John Swinbank, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.