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

Include a standard PSF as a Filter parameter #109

Open
aboucaud opened this issue Mar 31, 2022 · 6 comments
Open

Include a standard PSF as a Filter parameter #109

aboucaud opened this issue Mar 31, 2022 · 6 comments

Comments

@aboucaud
Copy link
Collaborator

As proposed by @herjy, in future releases of galcheat we could try to add a standard PSF to the parameters.

The format is still to be discussed : small image, parametric representation, optical wavefront.

The important point is that is should be consistent across surveys.

@herjy
Copy link

herjy commented Mar 31, 2022

An easy way to do this to begin with is probably to use analytic profiles with galsim as @thuiop suggested I think.
For now it's besides the point, but I don't think all PSFs have to use the same model or type of models, as long as the API remains the same no matter what survey we use. e.g. I would see no problem with calling something like
p_hsc_r = HSCSurvey.r.psf(ra, dec, t) # For HSC r-band PSF
p_hst_f814w = HSTSurvey.f814w.psf(ra, dec, t) # For HST f814w PSF
One calls a parametric profile, but the other uses an image and a distortion under the hood, that would be completely fine with me. I put ra, dec and t, but I don't anticipate that we'd provide full time and space coverage for varying PSFs, though if the information is available and can be used in galcheat, why not. And if not, we should just let the user know that the psf used is static.

@herjy
Copy link

herjy commented Mar 31, 2022

I'll try to make a proposition there.

@ismael-mendoza
Copy link
Contributor

Would the BTK default PSF be approrpiate for this ? ( at least for the constant version)

@herjy
Copy link

herjy commented Mar 31, 2022

Right, actually BTK already has everything we need for that. This is making me reconsider. Should we leave it at that or provide what is basically the get_psf method from BTK here?
I think that's the kind of thing that has its place in galcheat but we should think of what's the easiest.

@ismael-mendoza
Copy link
Contributor

Maybe we can discuss this in the BTK meeting this Wednesday, I hesitate to include simulations from galsim in galcheat since it feels out of scope for a "cheatsheet".

@ismael-mendoza
Copy link
Contributor

Axel suggests: provide psf parameters / format understood by galsim so that galsim is not a dependency. See how they do it in https://github.com/LSSTDESC/psf-weather-station

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

No branches or pull requests

3 participants