python-bloomfilter icon indicating copy to clipboard operation
python-bloomfilter copied to clipboard

Set correct count after unioning two bloom filters

Open grafke opened this issue 8 years ago • 2 comments

After unioning two bloom filters, the count of the resulting bloom filter was set to 0. I updated the union method to update the self.count of the result as well.

grafke avatar Dec 07 '16 10:12 grafke

This sets the correct count for union, but will need to also have it set for intersection in this case. I can take a look at adding this soon.

Another possibility is removing the count entirely and moving to a more pressure based system for growing the filter that isn't exposed to the API. Let me think about this some more.

jaybaird avatar Dec 09 '16 17:12 jaybaird

I added the correct count in the intersection method as well.

grafke avatar Dec 12 '16 15:12 grafke