Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Iterative PSF fitting routine where all newly found sources are re-fit along with previously detected sources. #732

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Onoddil
Copy link
Contributor

@Onoddil Onoddil commented Oct 3, 2018

An initial inclusion of a more robust PSF fitting algorithm, similar in scope to IterativelySubtractedPSFPhotometry. However, instead of fitting sources, subtracting them and fitting sources found in the residual image, IterativeGroupPSFPhotometry looks at the residual image for new sources, and then re-fits all sources in the subsequent loop.

The loop then becomes
0) [find or provide initial sources]
1a) group sources
1b) fit sources in main image
1c) compute residual
1d) find new sources in residual
1e) [stop if no new sources]
2a) re-group sources
2b) re-fit sources in main image
2c) compute new residual
2d) find further sources in residual
...

This allows for the case where individual fitting of one PSF model to two overlapping sources would give unphysical parameters for the brightest source, providing a "second chance" to correct the derived parameters of the first source with the knowledge that there is a second source, previously hidden in the image. Because the fitting always occurs on the image, rather than the previous iteration's residuals, we do not propagate any poor fits from any prior loops forward.

@Onoddil Onoddil changed the title Extended Iterative PSF Fitting Routine Iterative PSF fitting routine where all newly found sources are re-fit along with previously detected sources. Oct 12, 2018
…found sources along with any subsequently detected sources within image residuals.
Base automatically changed from master to main March 16, 2021 02:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants