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

afocal systems break focal length and platescale calculations #200

Open
mperrin opened this issue Aug 23, 2018 · 2 comments
Open

afocal systems break focal length and platescale calculations #200

mperrin opened this issue Aug 23, 2018 · 2 comments
Labels
fresnel Issues related to the Fresnel propagation part of the code
Milestone

Comments

@mperrin
Copy link
Collaborator

mperrin commented Aug 23, 2018

Issue by douglase
Tuesday Jan 03, 2017 at 18:02 GMT
Originally opened as mperrin/poppy#200


As discovered recently by @neilzim and as previously noted but not documented, the current way plate scales are calculated in the fresnel wavefront class depends on the focal length attribute carried through the sequence of propagations by the wavefront class. Thus, it breaks when you have an afocal system where plane-to-plane propagation occurs.

We need either another way of carrying focal length through a system, or more graceful failure and better documentation.

@mperrin mperrin added the fresnel Issues related to the Fresnel propagation part of the code label Nov 27, 2018
@mperrin mperrin added this to the Winter 2019 release milestone Aug 12, 2019
@mperrin mperrin modified the milestones: Winter 2019 release, 0.9.2 Jun 19, 2020
@mperrin mperrin modified the milestones: 1.0.0, 1.1.0 Feb 4, 2021
@isa-kain
Copy link

Hi @mperrin! I'm running into this bug as well, where the focal length is "forgotten" in collimated space and the plate scale becomes -inf. Has there been any progress on this, or do you have a suggested quick fix?

@mperrin
Copy link
Collaborator Author

mperrin commented Jun 5, 2023

@isa-kain Thanks for your patience & apologies for not being able to write back sooner. I have not had a chance to look at this yet.

It might help if you could share some example code that runs into this bug, maybe a simple simulation of an focal simulation like this in a script or notebook. Having a runnable code example would help me get started on looking into this. Cheers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fresnel Issues related to the Fresnel propagation part of the code
Projects
None yet
Development

No branches or pull requests

2 participants