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

Add meas_extensions_trailedSources to lsst_distrib

    XMLWordPrintable

Details

    • RFC
    • Status: Implemented
    • Resolution: Done
    • DM
    • None

    Description

      meas_extensions_trailedSources is a measurement plugin that characterizes fast-moving, trailed sources. Mainly, it aims to measure the length, angle, and flux. The current implementation involves two main algorithms: 

      1. Naive: Utilizes previously measured second moments to estimate the length and angle of a trail, and compute the end points. This model is inexpensive to compute, but under-estimates length. 
      2. Vereš: A chi-squared minimization using the analytic model from Vereš et al. 2012 – an axisymmetric Gaussian convolved with a line. Uses the Naive measurements and centroid as the initial parameter guess. This model assumes the flux in a given pixel is the value of the model at the center of that pixel. It estimates true length better, but is more computationally expensive than the Naive model. It also adds a flux, and centroid measurement. 

      Further development is ongoing. A correction to the Naive measurement is being investigated as well a third algorithm to provide a full forward model for high signal-to-noise trails. The overall goal is to provide a 'smart' plugin that runs each algorithm sequentially, base on signal-to-noise. Ie. For low signal-to-noise, the precision of the Naive measurement will be suitable. If the signal-to-noise is higher, then the next algorithm will be run, and so on.

      The package is currently available at lsst/meas_extensions_trailedSourceslsst-dm/meas_extensions_trailedSources. It includes unit tests for each measurement algorithm and is able to be setup along side lsst_distrib. Only dependencies outside of lsst_distrib are scipy.optimize and numpy.

      Attachments

        Issue Links

          Activity

            ktl Kian-Tat Lim added a comment - Flagging per https://developer.lsst.io/communications/rfc.html#standard-procedures-that-require-an-rfc as "Adding a new package to the build".

            Nice, this sounds like a useful addition. Feel free to reach out if you want some DECam images with real satellite trails to test it on. Please consider adding some meaningful content to the doc directory before you add this package to the build.

            mrawls Meredith Rawls added a comment - Nice, this sounds like a useful addition. Feel free to reach out if you want some DECam images with real satellite trails to test it on. Please consider adding some meaningful content to the doc directory before you add this package to the build.
            tjenness Tim Jenness added a comment -

            DMCCB have no problem with adding this package to lsst_distrib but we request that there is a code review by science pipelines before it is added.

            tjenness Tim Jenness added a comment - DMCCB have no problem with adding this package to lsst_distrib but we request that there is a code review by science pipelines before it is added.
            tjenness Tim Jenness added a comment -

            Remember also to add the standard lint.yaml GitHub action.

            tjenness Tim Jenness added a comment - Remember also to add the standard lint.yaml GitHub action.
            tjenness Tim Jenness added a comment -

            langfzac can you please decide if you are going to adopt this RFC? Its review period has expired and the DMCCB are happy for this RFC to be adopted. Please mark it as adopted and link an "is triggering" ticket to do the implementation work.

            tjenness Tim Jenness added a comment - langfzac can you please decide if you are going to adopt this RFC? Its review period has expired and the DMCCB are happy for this RFC to be adopted. Please mark it as adopted and link an "is triggering" ticket to do the implementation work.

            Yes. Thanks, Tim! (Still new to this process).

            langfzac Zach Langford [X] (Inactive) added a comment - Yes. Thanks, Tim! (Still new to this process).

            People

              langfzac Zach Langford [X] (Inactive)
              langfzac Zach Langford [X] (Inactive)
              Eric Bellm, Ian Sullivan, John Parejko, Kian-Tat Lim, Mario Juric, Meredith Rawls, Tim Jenness, Zach Langford [X] (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                Jenkins

                  No builds found.