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

Fix PSFEx in good seeing

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:

      Description

      From the recent HSC data we know that the performance of the PSF modeling code, PSFEx, is poor when the seeing is < ~0.45". Robert Lupton found that the single pixel basis functions of the PSF model were being Lanczos interpolated. He updated the code to use the overlap of model basis with the original pixels. This ticket implements his change.

        Attachments

        1. data.png
          data.png
          70 kB
        2. new_sims.png
          new_sims.png
          69 kB
        3. old_sims.png
          old_sims.png
          50 kB
        4. vvds_rho1_1am.png
          vvds_rho1_1am.png
          68 kB
        5. vvds_rho1.png
          vvds_rho1.png
          57 kB
        6. vvds_size_res_vs_seeing.png
          vvds_size_res_vs_seeing.png
          65 kB

          Activity

          Hide
          price Paul Price added a comment -

          The docstring for vignet_resample_pixel needs to be updated after copy/paste.

          Show
          price Paul Price added a comment - The docstring for vignet_resample_pixel needs to be updated after copy/paste.
          Hide
          rearmstr Bob Armstrong added a comment -

          Good catch. I have updated the docstring.

          Show
          rearmstr Bob Armstrong added a comment - Good catch. I have updated the docstring.
          Hide
          price Paul Price added a comment -

          Is this ready to merge?

          Show
          price Paul Price added a comment - Is this ready to merge?
          Hide
          rearmstr Bob Armstrong added a comment -

          Yes, I will merge this morning.

          Show
          rearmstr Bob Armstrong added a comment - Yes, I will merge this morning.
          Hide
          rearmstr Bob Armstrong added a comment -

          Merged

          Show
          rearmstr Bob Armstrong added a comment - Merged

            People

            • Assignee:
              rearmstr Bob Armstrong
              Reporter:
              rearmstr Bob Armstrong
              Reviewers:
              Paul Price
              Watchers:
              Bob Armstrong, Paul Price, Robert Lupton
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: