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

Add len support for unmodified catalogs #333

Closed
smcguire-cmu opened this issue Aug 9, 2024 · 0 comments · Fixed by #371
Closed

Add len support for unmodified catalogs #333

smcguire-cmu opened this issue Aug 9, 2024 · 0 comments · Fixed by #371
Assignees
@camposandro
Labels
enhancement New feature or request

Comments

@smcguire-cmu
Copy link
Contributor

smcguire-cmu commented Aug 9, 2024 •
edited
Loading

Feature request

We can get the length of a catalog after it has been loaded and before any filtering operations, which some users may find useful. We can use the len for this and return None or make a warning if a catalog has been filtered.
@smcguire-cmu smcguire-cmu added the enhancement New feature or request label Aug 9, 2024
@smcguire-cmu smcguire-cmu changed the title Add len support for unmodified catalogs Add len support for unmodified catalogs Aug 9, 2024
@camposandro camposandro mentioned this issue Oct 7, 2024
Merged
5 tasks
@camposandro camposandro self-assigned this Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@camposandro camposandro

Labels
enhancement New feature or request
Projects
HATS / LSDB
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Override catalog's __len__ method astronomy-commons/hats
2 participants
@camposandro @smcguire-cmu