Matt Abrams
Matt Abrams
I do recommend upgrading to HLL+ it provides some memory efficiencies over the original HLL implementation. The number of entries in a HLL (or HLL+) has very little impact on...
Excellent. Thanks for sharing. I'd like to see this implemented in stream-lib and will look to do so in the next few months unless someone beats me to it. On...
working my way through testing this one. thanks for the patch.
I'm OK with API breaking changes in this release if its the right thing to do.
OK, thanks I'll wait till you are ready with the other patch before merging this one. Code looks good though and passes tests.
Simon - Thanks! - HyperLogLog:151,161 seems to have a redundant "+ 1", perhaps appeared on copy-paste from LogLog, suggested correction: final int r = Long.numberOfLeadingZeros((hashedValue with and stumbled over a...
This looks like a nice improvement. Can you add a test that would have failed without the check? We aren't sure that the additions you made to the test would...
I'm not sure. Stream-lib's CountMinSketch has fallen behind what @tdunning has in his repo. Hopefully we can work on pulling in a more recent version. Otherwise we may need to...
Sorry @tdunning. My memory seems to be failing me... Great news on new releases of T-digest and FloatHistogram. @alexandrnikitin I'll look into this and #140 as soon as I can.
Confirmed that it returns true. This behavior doesn't actually result in incorrect cardinality results but it does violate the interface's contract. The issue is that for efficiency purposes we do...