Avoid zeros in dice and iou

[Borda]

What does this PR do?

Fixes #2545 better way that adding epsilon

PR review

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

[williamFalcon]

it really should be an epsilon...

but in 16 bit 1e-15 is 0.... @Borda

[codecov]

Codecov Report

Merging #2567 into master will increase coverage by 0%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #2567   +/-   ##
======================================
  Coverage      91%     91%           
======================================
  Files          70      70           
  Lines        5724    5753   +29     
======================================
+ Hits         5215    5242   +27     
- Misses        509     511    +2     

[Borda]

it really should be an epsilon...
but in 16 bit 1e-15 is 0.... @Borda

if you test that denom is not zero then you do not need any eps

EDIT: https://stackoverflow.com/a/9528486/4521646

import sys
sys.float_info.epsilon
2.220446049250313e-16

[pep8speaks]

Hello @Borda! Thanks for updating this PR.

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2020-07-09 22:45:24 UTC