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.

Sign up for GitHub

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

Jump to bottom

Suggestions for improved MFT and SAM tests #29

Closed
1 of 2 tasks
mperrin opened this issue Aug 23, 2018 · 2 comments
Closed
1 of 2 tasks

Suggestions for improved MFT and SAM tests #29

mperrin opened this issue Aug 23, 2018 · 2 comments
Labels
enhancement New feature or request testing

Comments

@mperrin
Copy link
Collaborator

mperrin commented Aug 23, 2018

Issue by mperrin
Tuesday Dec 16, 2014 at 17:00 GMT
Originally opened as mperrin/poppy#29

Suggestions from Laurent:

  1. Test the MFT function by setting up a calculation that uses the MFT to produce a PSF on the exact same sampling as an FFT. These should be identical to near machine precision.
  • Done
  1. Test the overall SAM implementation by setting up a calculation of a band limited coronagraph on a square aperture, for which analytic solutions exist. Get these from Laurent since he already has this info organized somewhere. Compare numerical results to the analytic solution.
  • TBD
@mperrin
Copy link
Collaborator Author

mperrin commented Aug 23, 2018

Comment by mperrin
Saturday Jan 24, 2015 at 01:46 GMT

MFT/FFT exact equivalence tests added as part of resolving mperrin/poppy#38

@mperrin mperrin added enhancement New feature or request testing labels Aug 23, 2018
@mperrin
Copy link
Collaborator Author

mperrin commented Apr 2, 2021

No further work needed on this at this time.

@mperrin mperrin closed this as completed Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mperrin