Marshall Perrin

Results 178 comments of Marshall Perrin

Hi, I can advise more with this tomorrow. Russell no we don’t expect users to install the dev version; rather the way this works is poppy and WebbPSF have synchronized...

OK, that error is due to a bug in synphot, in which prior versions are not compatibly with astropy 5.0. The required fix is to have the most recent synphot,...

also - [ ] remove the back ported version of `lru_cache` in the `vendor` directory.

and update the documentation and setup config to enforce a minimum Python version of 3.4.

**Comment by [mperrin](https://github.com/mperrin)** _Tuesday Aug 29, 2017 at 19:44 GMT_ ---- @kvangorkom This PR may be of interest to you. (but definitely feel free to completely ignore this if you're...

**Comment by [mperrin](https://github.com/mperrin)** _Tuesday Aug 29, 2017 at 19:48 GMT_ ---- Oh, and this PR also adds functions `plot_noll_zernikes` and `plot_wyant_zernikes` to make pretty plots of the polynomials. Mostly for...

**Comment by [kvangorkom](https://github.com/kvangorkom)** _Tuesday Aug 29, 2017 at 20:13 GMT_ ---- I put together some code a few months ago to do something very similar! I managed to cobble together...

**Comment by [mperrin](https://github.com/mperrin)** _Tuesday Aug 29, 2017 at 20:32 GMT_ ---- oh nice! I figured there had to be an algorithmic way to do that mapping rather than my hack...

**Comment by [kvangorkom](https://github.com/kvangorkom)** _Tuesday Aug 29, 2017 at 20:56 GMT_ ---- I've seen both index 1 and index 0, and I'm not sure there's a preferred convention. (I believe both...

**Comment by [coveralls](https://github.com/coveralls)** _Tuesday Aug 29, 2017 at 21:35 GMT_ ---- [![Coverage Status](https://coveralls.io/builds/13040843/badge)](https://coveralls.io/builds/13040843) Coverage decreased (-0.9%) to 64.53% when pulling **40ac0269957f2c24f5db4c28e0f777f888efbb7d on fringe_zernike** into **569d5c60f7cf86a256c661227bd6269de4144005 on master**.