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.

Sign up for GitHub

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

Jump to bottom

Sources order in BasicPSFPhotometry #645

Open
abeelen opened this issue Feb 5, 2018 · 0 comments · May be fixed by #746
Open

Sources order in BasicPSFPhotometry #645

abeelen opened this issue Feb 5, 2018 · 0 comments · May be fixed by #746
Labels
feature request psf

Comments

@abeelen
Copy link

abeelen commented Feb 5, 2018

When using BasicPSFPhotometry with Fixed Centroids the returned list of sources do not follow the same order as the input init_guesses, but rather the grouping order (thanks to the hstack at the end of the function), this could lead to confusion.

I would suggest returning the results using the same order, or to place a clear warning in the documentation and/or docstring.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request psf
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added flag to BasicPSFPhotometry to preserve ID order Onoddil/photutils
3 participants
@abeelen @larrybradley @mirca