trippy icon indicating copy to clipboard operation
trippy copied to clipboard

Desired feature: MCMCfit only fit amplitude (not XY)

Open Mikea1985 opened this issue 6 years ago • 2 comments

Sometimes I'm confident in the centroiding (from SExtractor or whatever) and just want to fit the amplitude so that I can make an image with the object subtracted out, and don't want to wait for the time-intensive 3-step MCMC fit of xy, amplitude, xy+amplitude. Also, sometimes I find that the MCMC fit runs off into lalaland and doesn't actually centroid on the source (not sure what's up with that), in which case the fit isn't useful. It would be really nice if there was a way to ask MCMCfit to only fit the amplitude, maybe with a fitXY=True/False keyword.

I tried implementing this, but failed misserably.

Mikea1985 avatar Dec 05 '18 07:12 Mikea1985

This could be implemented with small changes to MCMCfit and the realization that lnprob can fit xym, only xy, or only m. With appropriately exposed options in the MCMCfit call, this wouldn't be too difficult. Will work on it

fraserw avatar Dec 18 '19 22:12 fraserw

I think my Pull Request #36 actually fixes this old issue.

Mikea1985 avatar Jan 22 '20 17:01 Mikea1985