P. L. Lim

Results 538 comments of P. L. Lim

7 years on, @svolfman and I were just being very confused why the search area isn't a circle until she dug out this issue! xref spacetelescope/jdaviz#1413

We need to compare this with what is already established in `imexam`, see https://github.com/spacetelescope/imexam/blob/b3975733e0ef459fc7d137acec9bf4aa2865d6c5/imexam/imexamine.py#L1266 . It also uses `photutils` for photometry.

I am guessing this is going to be a blocker for both spacetelescope/jdaviz#1048 and spacetelescope/jdaviz#1049 ? Can we even use this without a way to translate between `regions` region and...

@eteq , we discussed this at Indigo tag-up today. I modified your original post with our plan. Please look at it and let us know if you have questions or...

Is this really in scope for "astropy tutorials" per se? Do we want to turn this into a computer science course?

Cross-list is lower barrier.

I am seeing performance issues as well when I run the `BasicPSFPhotometry` step in https://github.com/spacetelescope/da5-notebooks/blob/master/notebooks/psf_grid_simulation_photometry/GriddedPSFModel_simulation_and_photometry.ipynb . I was using 512x512 image with 25 stars and a `fitshape` of `(101, 101)`....

@bsipocz , did you try to profile it? Is the bottleneck at least different now, after refactoring by @perrygreenfield ?

`add_array` and `extract_array` are from [astropy.nddata.utils](https://github.com/astropy/astropy/blob/master/astropy/nddata/utils.py). 😬

Thanks for the detailed report, @Onoddil ! If the indices and copy are unavoidable, can also think about using Dask or multiprocessing. Cython crossed my mind too though I am...