search

FastFilter / fastfilter_cpp

Fast Approximate Membership Filters (C++)
Apache License 2.0
261 stars 24 forks source link

Reorganize the code: use SimpleMixSplit instead of TwoIndependentMultiplyShift #18

Closed thomasmueller closed 3 years ago

thomasmueller commented 3 years ago

It's a bit hard to integrate more algorithms currently. I would like to add the ribbon filter for example, and maybe more will be added later. I will try to re-organize the code a bit.

thomasmueller commented 3 years ago

Sometimes we use TwoIndependentMultiplyShift; we should probably use SimpleMixSplit consistently.

lemire commented 3 years ago

Sometimes we use TwoIndependentMultiplyShift; we should probably use SimpleMixSplit consistently.

Yes. I agree.

thomasmueller commented 3 years ago

I included some more changed I had in my working directory... I hope it still works. I want to change some more stuff.