world-federation-of-advertisers / cardinality_estimation_evaluation_framework

Evaluation framework and methods for estimating cardinalities of groups of sets
Apache License 2.0
22 stars 9 forks source link

[Part 2]Fix bug: Remove incompatible estimator noiser for the HLL. #112

Closed huangxichen1 closed 3 years ago

huangxichen1 commented 3 years ago

This PR fix the bug in the test/interoperability_test.py by removing the un-used and buggy estimate noiser.

IMPORTANT: Please merge the [Part X]Fix bug... PR in the reverse order, as they are dependent. Eg, merge [Part 3]Fix bug into [Part 2]Fix bug first, then [Part 2]Fix bug into [Part 1]Fix bug.