Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PSF stamp size large #10

Open
laldoroty opened this issue Jul 16, 2024 · 1 comment
Open

PSF stamp size large #10

laldoroty opened this issue Jul 16, 2024 · 1 comment
Assignees

Comments

@laldoroty
Copy link
Collaborator

In phrosty.imagesubtraction.get_imsim_psf():

psf = config.getPSF_Image(501, x=pxradec[0], y=pxradec[1]).array.T 

This says that the stamp size of the retrieved PSF from roman_imsim.utils will be 501 x 501 px.

Dan says Roman project won't know the PSF beyond ~30x30. So:

  1. Why so large? (ask Lei)
  2. Can we make it smaller?
@laldoroty laldoroty self-assigned this Jul 16, 2024
@laldoroty
Copy link
Collaborator Author

laldoroty commented Jul 17, 2024

Lei says "it is a little bit abitary. but since space telescope psf usually has a long tail/spikes. we want to preserve the whole structure. so the size should be large enough. also, we found any discontinuity in psf kernel can result in some undesired features to diff. The problem is the truncation at the kernel boundary. if you can make sure there is no significant truncation after you reduce the size, that's fine I think"

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant