pytim
pytim copied to clipboard
Update sanity_check.py
fixes a double call to guess radii, which creates trouble
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 85.14%. Comparing base (
82b655e
) to head (b90f41b
). Report is 42 commits behind head on master.
:exclamation: Current head b90f41b differs from pull request most recent head 49c6764. Consider uploading reports for the commit 49c6764 to get more accurate results
Additional details and impacted files
@@ Coverage Diff @@
## master #325 +/- ##
==========================================
+ Coverage 85.05% 85.14% +0.08%
==========================================
Files 38 38
Lines 3005 2996 -9
==========================================
- Hits 2556 2551 -5
+ Misses 449 445 -4
Files | Coverage Δ | |
---|---|---|
pytim/sanity_check.py | 76.83% <ø> (+1.28%) |
:arrow_up: |
... and 6 files with indirect coverage changes
Continue to review full report in Codecov by Sentry.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 0baf046...49c6764. Read the comment docs.
@balazsfabian, this was left pending, do you remember why it was not merged? I have some vague memory of you mentioning that it was not a good idea to remove that double call.