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

Add getContainerCount() method #611

Open
lemire opened this issue Dec 25, 2022 · 2 comments
Open

Add getContainerCount() method #611

lemire opened this issue Dec 25, 2022 · 2 comments

Comments

@lemire
Copy link
Member

lemire commented Dec 25, 2022

Is your feature request related to a problem? Please describe.
In some instances, we want to sort bitmaps with respect to the number of containers they hold. See #608 (comment)

Describe the solution you'd like
We should add a function to our public API that gives convenient access to the number of containers.

It would be as trivial as return highLowContainer.size(); though we want to cover both RoaringBitmap and ImmutableRoaringBitmap (and maybe the 64-bit bitmaps as well?). Proper documentation should be included.

@sladyn98
Copy link

@lemire Any good places to start ? I would like to contribute to this issue

@lemire
Copy link
Member Author

lemire commented Apr 1, 2023

@sladyn98 Good! Look at how we compute the cardinality. Computing the number of containers is similar/simpler.

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

2 participants