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

Implement convolution in the general setting #6

Open
1 of 3 tasks
b-remy opened this issue Feb 2, 2022 · 1 comment
Open
1 of 3 tasks

Implement convolution in the general setting #6

b-remy opened this issue Feb 2, 2022 · 1 comment

Comments

@b-remy
Copy link
Owner

b-remy commented Feb 2, 2022

This issue is to document the implementation of the convolution operation against galsim.

So far we demonstrated that the convolution was working well only considering Gaussian profiles and even stamp size.

Here is a list of what I need to implement:

  • odd stamp size convolution
  • Fourier image resampling
  • Inverse Fourier transform with kwrapping
@b-remy
Copy link
Owner Author

b-remy commented Feb 2, 2022

And I just managed to handle odd stamp size for the convolution. It is also a step further for DifferentiableUniverseInitiative/GalFlow#8 .

Capture d’écran 2022-02-03 à 00 13 36

The demo is here: https://colab.research.google.com/drive/1bSHE1YVZMnmLGucnYpbeFxdw2EL7x8Jy?usp=sharing
this nb is a mess, but I am going to open a PR with unit test ;-)

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