mperrin / webbpsf

James Webb Space Telescope PSF simulation tool - NOTE THIS VERSION OF REPO IS SUPERCEDED BY spacetelescope/webbpsf
BSD 3-Clause "New" or "Revised" License
16 stars 15 forks source link

For coronagraph masks, add "bar_offset_y" for NIRCam. Add similar for the round occulters and MIRI as well. #191

Open mperrin opened 6 years ago

mperrin commented 6 years ago

As requested by @ariedel for use in Pandeia ETC coronagraph generation script

ariedel commented 6 years ago

Use case is that Pandeia requires the PSF to be centered on a pixel; this currently requires us to snap to pixel positions rather than the actual requested XY location.

mperrin commented 6 years ago

This turns out to be a real pain in the neck for MIRI, given the different rotated coordinate systems. Because of how the optics are implemented, the "natural" way to offset the FQPM is in V2V3, rotated by ~ 5 deg from the instrument detector X and Y (which is what Pandeia wants to work in). So even implementing a "simple" option to offset the focal plane mask by a given amount inevitable is going to involve some coordinate transforms. Not hard, just more work.

NIRCam's comparatively simple.

mperrin commented 6 years ago

@ariedel, remind me where we left things with the coronagraph PSF generation script. I have this open issue about adding an option for shifting the masks around, and added code back in February to implement it. I'm not sure if any more is needed now? If you're happy with the coronagraph PSF generation script then good and I won't worry about any more effort on this issue.