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

Add meas_extensions_photometryKron to lsst_distrib

    XMLWordPrintable

    Details

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

      Description

      meas_extensions_photometryKron provides a measurement algorithm which is returns the flux in an elliptical aperture as defined by Kron (1980): the Kron radius is measured as the moment within an (elliptical) aperture of six times the (adaptive) moment, and the Kron flux is measured within an (elliptical) aperture of twice the Kron radius.

      This package is currently available, and can be built using lsstsw. I propose including it in lsst_distrib, so that it can be installed more easily and will automatically be included in CI.

      The package has no dependencies which are not currently satisfied within lsst_distrib.

        Attachments

          Issue Links

          is triggering

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-3440 add meas_extensions_photometryKron to lsstsw, lsst_distrib

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-6615 Please include meas_extensions_photometryKron in lsst_apps

          • Won't Fix

            Activity

            Descending order - Click to sort in ascending order
            Hide
            Permalink
            swinbank John Swinbank added a comment -

            I've seen no objections to this. As part of the implementation, we'll make sure there's some minimal documentation.

            Show
            swinbank John Swinbank added a comment - I've seen no objections to this. As part of the implementation, we'll make sure there's some minimal documentation.
            Hide
            Permalink
            wmwood-vasey Michael Wood-Vasey added a comment -

            +1

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - +1
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Given the code exists, builds fine and follows our dependencies and is code that we will support I don't see any issues with including it in lsst_distrib. What's the documentation situation?

            Show
            tjenness Tim Jenness added a comment - Given the code exists, builds fine and follows our dependencies and is code that we will support I don't see any issues with including it in lsst_distrib . What's the documentation situation?

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Watchers:
              John Swinbank, Michael Wood-Vasey, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End: