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

Include IDs in Star (centroid) objects #90

Open
markasoftware opened this issue Mar 4, 2023 · 0 comments
Open

Include IDs in Star (centroid) objects #90

markasoftware opened this issue Mar 4, 2023 · 0 comments

Comments

@markasoftware
Copy link
Member

Keeping track of centroid indices is a PITA in star-id algorithms right now. For example, if you have a function that needs to operate on a subset of the centroids, it is currently necessary to pass the full list of centroids, and then another list of centroid indices that you want to work on, instead of just passing a list of centroids (or pointers to centroids), which would be cleaner.

(Similarly, it might be good to include indices in CatalogStars after narrowing. Yes, we have names right now, but those names don't allow fast lookup into the full Catalog).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant