data61 / clkhash

CLK hash: hash pii for entity matching

Apache License 2.0

47 stars 9 forks source link

fix deprecation warning, reduced hypothesis test cases range to somet… #391

Closed wilko77 closed 4 years ago

wilko77 commented 4 years ago

…hing more sensible.

Testing with such huge ranges for threshold distance and candidates leads to some floating point weirdness. I think we can just test for numbers up to +/-1e20. That should cover most realistic use cases.

codecov[bot] commented 4 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3bafb67). Click here to learn what that means. The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #391   +/-   ##
=========================================
  Coverage          ?   94.65%           
=========================================
  Files             ?       12           
  Lines             ?      936           
  Branches          ?        0           
=========================================
  Hits              ?      886           
  Misses            ?       50           
  Partials          ?        0