-
Notifications
You must be signed in to change notification settings - Fork 182
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 benches for ZeroMap #853
Comments
We should probably write the same benches for |
@andrewnguyen is interested in working on this |
(Andrew isn't planning on fixing this anymore) |
Alan plans to work on this |
It's not clear to me what these benches are doing. I've put together a basic I see with |
@poulsbo it's worth comparing against HashMap and also potentially LiteMap. We absolutely expect that lookup is more expensive, the question to answer is "how much more expensive" :) utils/litemap already has a couple benchmarks to copy. The main operations we care about are read operations, but benchmarking some write operations can be good too. Note that borrowed ZeroMaps will switch internal implementations if you try writing to them, it would be interesting to capture the performance impact of that. |
I would like to have results to allow us to expand this table to include ZeroMap. |
Follow-up from #652
Action items:
@andrewnguyen
The text was updated successfully, but these errors were encountered: