data61 / clkhash

CLK hash: hash pii for entity matching
Apache License 2.0
47 stars 9 forks source link

read clks in a streaming fashion #673

Closed wilko77 closed 1 year ago

wilko77 commented 1 year ago

Not sure about the API change. Maybe introduce an extra function?

However, this way of doing things is:

codecov[bot] commented 1 year ago

Codecov Report

Merging #673 (c943981) into main (d606c9a) will decrease coverage by 2.21%. The diff coverage is 72.41%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #673 +/- ## ========================================== - Coverage 94.42% 92.21% -2.21% ========================================== Files 13 13 Lines 969 1028 +59 ========================================== + Hits 915 948 +33 - Misses 54 80 +26 ```
wilko77 commented 1 year ago

superseded by #675