search

dice-group / dice-hash

https://dice-group.github.io/dice-hash/
GNU Affero General Public License v3.0
2 stars 1 forks source link

add rapidhash #48

Open bigerl opened 2 months ago

bigerl commented 2 months ago

There seems to be a improved version of wyhash. We should add it as a new hashing scheme.

https://github.com/Nicoshev/rapidhash

Please also test how it performs in https://github.com/martinus/map_benchmark

Interesting for us are especially tsl::sparse_map and ankerl::unordered_dense in the configurations and it's segmented version, as well as with the bucket types standard and big.

tigerwagh commented 2 months ago

Hi,

I came across the issue regarding the integration of the improved version of wyhash and its performance testing with tsl::sparse_map and ankerl::unordered_dense in various configurations.

I am interested in contributing to this enhancement and would like to start working on it. Could you please confirm if the issue is still open and if I can proceed with the implementation and benchmarking?

bigerl commented 2 months ago

Hi @tigerwagh, the issue is open and has not been picked up yet. A contribution would be very appreciated. For a contribution please fork the repo and make a pull request. Also, please remind me to change the repo license to MIT+BSD so that you can freely use it in any private projects you might have.

If you just want to benchmark it in https://github.com/martinus/map_benchmark, I would be happy if you would share your results and a link to the code here.