Replace LMLSQFitter with TRFLSQFitter

astropy / photutils

Astropy package for source detection and photometry. Maintainer: @larrybradley

https://photutils.readthedocs.io

BSD 3-Clause "New" or "Revised" License

250 stars 138 forks source link

Replace LMLSQFitter with TRFLSQFitter #1917

Closed larrybradley closed 1 month ago

larrybradley commented 1 month ago

The default stddev of Astropy's Gaussian1D model is bounded to be strictly positive, so the TRFLSQFitter should be used. Followup to https://github.com/astropy/photutils/pull/1899

I've opened a PR for Gaussian2D to have the same default bounds: https://github.com/astropy/astropy/pull/17137

Many thanks to @astrofrog for pointing this out!