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

Add plugins for approximating the PSF as Gaussians with ngmix

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Story Points:
      0
    • Epic Link:
    • Sprint:
      DRP X16-3, DRP F16-1
    • Team:
      Data Release Production

      Description

      Create a new "meas_extensions_ngmix" package, and add SFM and Forced plugins to approximate the PSF using a mixture of Gaussians using ngmix's E-M fitter.

      There is example code to actually run ngmix in this mode in ngmix's README.md, but the tricky part is probably figuring out how to extract the outputs and write them to records. If at all possible, we should use the same format as modelfit_ShapeletPsfApprox, so either algorithm could be used to feed CModel (and other ngmix algorithms, when modelfit_ShapeletPsfApprox is configured to use only zeroth-order fitting).

      This will also require figuring out the failure modes and defining the appropriate flags. Given that ngmix has no C++ interface, this work should be entirely Python, and hence it may be desirable to address DM-4009 somehow first.

      This issue is almost certainly too large in terms of story points, but anything smaller doesn't really have a very well-defined end-point. Maybe subtasks?

        Attachments

          Issue Links

            Activity

            Hide
            pgee Perry Gee added a comment -

            This is the container issue around the others: DM-6123,24, and 25

            Show
            pgee Perry Gee added a comment - This is the container issue around the others: DM-6123 ,24, and 25
            Hide
            jbosch Jim Bosch added a comment -

            Am I correct in assuming that there's nothing additional to review here beyond what is in the subtask issues?

            Show
            jbosch Jim Bosch added a comment - Am I correct in assuming that there's nothing additional to review here beyond what is in the subtask issues?
            Hide
            swinbank John Swinbank added a comment -

            Yes, that's what I've previously discussed with Perry.

            Show
            swinbank John Swinbank added a comment - Yes, that's what I've previously discussed with Perry.
            Hide
            jbosch Jim Bosch added a comment -

            Moving this to Done, as all of its child tasks are. Please reopen if I'm mistaken and there is something else for me to review here.

            Show
            jbosch Jim Bosch added a comment - Moving this to Done, as all of its child tasks are. Please reopen if I'm mistaken and there is something else for me to review here.
            Show
            swinbank John Swinbank added a comment - Mentioned in release notes: https://confluence.lsstcorp.org/display/DM/Data+Release+Production+WIP+F16+Release+Notes

              People

              • Assignee:
                pgee Perry Gee
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Jim Bosch
                Watchers:
                Jim Bosch, John Swinbank, Perry Gee
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: