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

Add meas_extensions_convolved to lsst_distrib

    XMLWordPrintable

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None
    • Location:
      This ticket.

      Description

      meas_extensions_convolved is a plugin for the meas_base measurement framework. It convolves images before measuring aperture fluxes. The convolution provides a rough match to the PSF in other observations, and so aperture measurements on the convolved image are useful for:

      1. Measuring the flux that would be obtained with a fiber spectrograph, by using an aperture equal to the size of the spectrograph fiber and measuring with multiple target seeings to allow interpolation of the flux at the seeing of the spectroscopic observation.
      2. Measuring the color of galaxies in variable seeing, by using a common aperture and target seeing in each band.

      This functionality is actively being used by the DRP team.

      I propose to add this package as a dependency of lsst_distrib so that it can benefit from regular CI and so that it is distributed with future stack releases.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            Link to the package?

            Show
            Parejkoj John Parejko added a comment - Link to the package?
            Show
            price Paul Price added a comment - https://github.com/LSST/meas_extensions_convolved/
            Hide
            Parejkoj John Parejko added a comment -

            Looks like it's main contents are one file and its test. Should we just merge it with some other package?

            Show
            Parejkoj John Parejko added a comment - Looks like it's main contents are one file and its test. Should we just merge it with some other package?
            Hide
            swinbank John Swinbank added a comment -

            I have no objection to merging it with something else, but I don't really understand the advantage. What are you hoping to gain?

            Show
            swinbank John Swinbank added a comment - I have no objection to merging it with something else, but I don't really understand the advantage. What are you hoping to gain?
            Hide
            tjenness Tim Jenness added a comment -

            If we are using it and need it and DRP team are supporting it I can't see any reason not to include it in lsst_distrib.

            Show
            tjenness Tim Jenness added a comment - If we are using it and need it and DRP team are supporting it I can't see any reason not to include it in lsst_distrib .
            Hide
            Parejkoj John Parejko added a comment -

            John Swinbank no expectation of gain, I'm just not clear on whether a small package like this should live by itself or be merged with something. I'm in favor of it coming in either way.

            Show
            Parejkoj John Parejko added a comment - John Swinbank no expectation of gain, I'm just not clear on whether a small package like this should live by itself or be merged with something. I'm in favor of it coming in either way.
            Hide
            swinbank John Swinbank added a comment -

            Personally, I'd love to see a lot fewer, bigger packages. But I think this (small) package is consistent with our existing conventions, and I'd rather pull it in to distrib without having to restructure it!

            Show
            swinbank John Swinbank added a comment - Personally, I'd love to see a lot fewer, bigger packages. But I think this (small) package is consistent with our existing conventions, and I'd rather pull it in to distrib without having to restructure it!
            Hide
            swinbank John Swinbank added a comment -

            Let's do it. Work to be done on DM-9394.

            Show
            swinbank John Swinbank added a comment - Let's do it. Work to be done on DM-9394 .

              People

              Assignee:
              swinbank John Swinbank
              Reporter:
              swinbank John Swinbank
              Watchers:
              John Parejko, John Swinbank, Paul Price, Tim Jenness
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End: