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

meas_base plugins for CModel magnitudes

    XMLWordPrintable

Details

    Description

      Create meas_base Plugins for single-frame and forced measurement that uses the model-fitting primitives in meas_multifit to implement SDSS-style CModel magnitudes, in which we fit an exp and dev model separately and then fit the linear combination with ellipse parameters held fixed.

      An old-style plugin has already been implemented on the HSC fork, and should be used as a guide; this issue involves adapting that implementation to meas_base and potentially cleaning it up a bit. Note that the HSC implementation cannot be transferred directly to the LSST side because the meas_algorithms APIs are slightly different on the two forks.

      Attachments

        Issue Links

          Activity

            jbosch Jim Bosch added a comment -

            Sending this to Bob Armstrong for review. He doesn't have a JIRA account yet, but he does have a GitHub account, and we'll do the review there anyway, and I'll paste a summary back here when we're done.

            jbosch Jim Bosch added a comment - Sending this to Bob Armstrong for review. He doesn't have a JIRA account yet, but he does have a GitHub account, and we'll do the review there anyway, and I'll paste a summary back here when we're done.
            jbosch Jim Bosch added a comment -

            Review completed on GitHub. Most comments were just requests for clarification.

            Biggest outcome was noticing an incorrect factor (2 instead of pi) in SoftenedLinearPrior normalization, which spurred an improvement in the unit test coverage for that class, uncovering a larger bug in prior evaluation. Neither of these problems would have affected the code as it is configured by default, but would have affected mode unusual configurations of the prior.

            jbosch Jim Bosch added a comment - Review completed on GitHub. Most comments were just requests for clarification. Biggest outcome was noticing an incorrect factor (2 instead of pi) in SoftenedLinearPrior normalization, which spurred an improvement in the unit test coverage for that class, uncovering a larger bug in prior evaluation. Neither of these problems would have affected the code as it is configured by default, but would have affected mode unusual configurations of the prior.

            People

              jbosch Jim Bosch
              jbosch Jim Bosch
              Jim Bosch
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Jenkins

                  No builds found.