search

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

update webbpsf jitter convolution options #133

Closed mperrin closed 7 years ago

mperrin commented 8 years ago

Before the next release, which will be the first one that has OPDs which don't include artificial defocus as a (not great) proxy for jitter, we should change the default values such that jitter convolution is enabled by default: 

self.options['jitter'] = 'Gaussian'
self.options['jitter_sigma'] = 0.007

Perhaps should update the jitter API to allow 0.007*units.arcsec for explicitness.

And we should add suitable tests for this to the WebbPSF test suite. And documentation of when it is and isn't appropriate (isn't: coronagraphy and slit or MSA spectroscopy).

mperrin commented 7 years ago

this was done at some point prior to the 0.6 release.