Fixed MacOS rounding error

[dvezinet]

Main changes:

tests01_geom/test_01_GG.py:

• test16_dist_los_vpoly:

  • Specified tolerances for np.allclose(atol=1e-12, rtol=1e-12), for clarity
  • Large discrepancy observed between MacOS and Linux on this particular unit test, for some specific cases

• _distance_tools.pyx:

  • c_sqrt() was applied on a quantity that was 0 within machine accuracy but negative (-2e-17)
  • Resulting in a NaN that was propagating in certain special cases
  • Fix: apply c_sqrt(c_abs()) for better robustness

Issues:

Fixes, in devel, issue #930

[pep8speaks]

Hello @dvezinet! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file tofu/tests/tests01_geom/test_01_GG.py:

Line 1199:80: E501 line too long (94 > 79 characters) Line 1228:80: E501 line too long (81 > 79 characters) Line 1236:80: E501 line too long (84 > 79 characters) Line 2025:33: W292 no newline at end of file

Comment last updated at 2024-05-29 19:07:36 UTC