search

LSSTDESC / rail_sklearn

RAIL algorithms that depend on scikit-learn.
MIT License
1 stars 0 forks source link

missed one of two things I needed to change to fix the nan issue #7

Closed sschmidt23 closed 1 year ago

sschmidt23 commented 1 year ago

I missed a second instance of dist that can still cause problems, it is fixed by adding another "TEENY" to that distance calculation

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02% :tada:

Comparison is base (c6a981a) 95.43% compared to head (b58e696) 95.45%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #7 +/- ## ========================================== + Coverage 95.43% 95.45% +0.02% ========================================== Files 3 3 Lines 197 198 +1 ========================================== + Hits 188 189 +1 Misses 9 9 ``` | [Files Changed](https://app.codecov.io/gh/LSSTDESC/rail_sklearn/pull/7?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC) | Coverage Δ | | |---|---|---| | [src/rail/estimation/algos/k\_nearneigh.py](https://app.codecov.io/gh/LSSTDESC/rail_sklearn/pull/7?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=LSSTDESC#diff-c3JjL3JhaWwvZXN0aW1hdGlvbi9hbGdvcy9rX25lYXJuZWlnaC5weQ==) | `100.00% <100.00%> (ø)` | |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.