BlueBrain / NeuroTS

Topological Neuron Synthesis
https://neurots.readthedocs.io/en/stable
Apache License 2.0
36 stars 5 forks source link

Fix: Relax soma/first point check #68

Closed arnaudon closed 1 year ago

arnaudon commented 1 year ago

If one has a soma located far from origin, the rtol may consider the point to be too close, hence its better to only use atol.

codecov[bot] commented 1 year ago

Codecov Report

Merging #68 (f77efda) into main (d4d5ecb) will not change coverage. The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #68   +/-   ##
=======================================
  Coverage   97.57%   97.57%           
=======================================
  Files          39       39           
  Lines        2148     2148           
  Branches      330      330           
=======================================
  Hits         2096     2096           
  Misses         35       35           
  Partials       17       17           
Flag Coverage Δ
pytest 97.57% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
neurots/generate/soma.py 100.00% <100.00%> (ø)