pybloomfiltermmap3 __len__(bf) misreports after open(filename)

len(bf) misreports after open(filename)

Open mizvyt opened this issue 5 years ago • 4 comments

Issue in detail here: https://github.com/axiak/pybloomfiltermmap/issues/31

A fix is referenced, but I'd like to still think over the options.

Jan 13 '20 04:01 mizvyt

this, #38, #31 might all be linked. I'll take a look.

Aug 23 '20 04:08 prashnts

I was able to reproduce this finally, and noticed this stays even after #40. (Using bf.approx_len gets us correct length...) but need to dig deeper.

I reckon the problem might be around self._bf.count_correct and self._bf.elem_count in the cython class.

Which begs the question: why not use approx_len all the time?

Sep 30 '20 17:09 prashnts

For reference, this is the patch:

https://github.com/axiak/pybloomfiltermmap/pull/47/files#diff-a16b2c8dba66561e6fdef61b6efe2b04

Sep 30 '20 17:09 prashnts

Can you not merge that patch into your repository and that publish the updated version to the pypi?

Dec 09 '20 22:12 nffdiogosilva