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

parity of forward & inverse MFT transforms #35

Closed
mperrin opened this issue Jan 15, 2015 · 2 comments
Closed

parity of forward & inverse MFT transforms #35

mperrin opened this issue Jan 15, 2015 · 2 comments
Assignees
Milestone

Comments

@mperrin
Copy link
Owner

mperrin commented Jan 15, 2015

Begin forwarded message:

From: Anand Sivaramakrishnan anand@stsci.edu
Subject: troubles with parity of forward & inverse mDFT transforms...
Date: January 15, 2015 at 2:42:48 PM EST

Hi Joseph & Marshall

I use matrixDFT from WebbPSF as a standalone (to try to unify my code with WebbPSF utilities), and I'm having some trouble with coordinate flips when transforming forwards, then back.

I start with a psf in this example, transform it forwards to visibilities, then I try to transform it backwards as well as forwards - but get a flipped PSF both times. Running "matrixDFT_test.py" in the attached tar file will do these things, and give you two convenient ds9 commands to cut'n'paste to see the results.

This is a bit worrying, as matrixDFT is at the core of many things I do.

I also include matrixDFT.py copied from WebbPSF, with a print statement to show that I'm actually using this file from the *_test.py driver.

Any comments or suggestions would be great! If I should be using a different 'entry point' into the dft (such as SFT) please do let me know.

(I don't see any straightforward way to attach the file attachment to a github issue, but we have it in our inboxes)

@mperrin mperrin changed the title potential issue with parity of forward & inverse MFT transforms parity of forward & inverse MFT transforms Jan 15, 2015
@mperrin mperrin added this to the 0.3 milestone Jan 15, 2015
@mperrin
Copy link
Owner Author

mperrin commented Jan 21, 2015

Reproduced by @josePhoenix from Anand's example code. Definitely looks like I missed this one somehow before.

@josePhoenix
Copy link
Collaborator

Really a POPPY issue, see mperrin/poppy#38 for remaining subtasks (basically writing test cases).

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

2 participants