Skip to content

Commit 33e8bdb

Browse files
committed
small documentation fixes per PR comments
1 parent 6edbfee commit 33e8bdb

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

python/src/count_wrapper.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -34,8 +34,8 @@ void bind_count_min_sketch(py::module &m, const char* name) {
3434
.def_static("suggest_num_buckets", &count_min_sketch<W>::suggest_num_buckets, py::arg("relative_error"),
3535
"Suggests the number of buckets needed to achieve an accuracy within the provided "
3636
"relative_error. For example, when relative_error = 0.05, the returned frequency estimates "
37-
"satisfy the 'relative_error' guarantee that never overestimates the weights by may "
38-
"underestimate hte weights by 5% of the total weight in the sketch. "
37+
"satisfy the 'relative_error' guarantee that never overestimates the weights but may "
38+
"underestimate the weights by 5% of the total weight in the sketch. "
3939
"Returns the number of hash buckets at every level of the sketch required in order to obtain "
4040
"the specified relative error.")
4141
.def_static("suggest_num_hashes", &count_min_sketch<W>::suggest_num_hashes, py::arg("confidence"),

python/tests/count_min_test.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,8 @@ def test_count_min_example(self):
6262
cm2.update(i, i)
6363
cm.merge(cm2)
6464

65-
# and the estimated weight for the meerged values should now be
66-
# at least 2x the value
65+
# and the estimated weight for the overlapped meerged values
66+
# (n/2 to n) should now be at least 2x the value
6767
self.assertGreaterEqual(cm.get_estimate(n), 2 * n)
6868

6969
# finally, serialize and reconstruct

0 commit comments

Comments
 (0)