edges-collab / edges-cal

Code to calibrate EDGES data
MIT License
0 stars 1 forks source link

Fix thermistor read #116

Closed steven-murray closed 1 year ago

steven-murray commented 1 year ago

Fixes small bug in reading thermistor

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 91.66% and project coverage change: -1.39 :warning:

Comparison is base (e57c622) 88.86% compared to head (ed5d9e3) 87.48%.

:exclamation: Current head ed5d9e3 differs from pull request most recent head 3c8bf22. Consider uploading reports for the commit 3c8bf22 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #116 +/- ## ========================================== - Coverage 88.86% 87.48% -1.39% ========================================== Files 15 15 Lines 2803 2812 +9 Branches 465 464 -1 ========================================== - Hits 2491 2460 -31 - Misses 188 225 +37 - Partials 124 127 +3 ``` | [Impacted Files](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab) | Coverage Δ | | |---|---|---| | [src/edges\_cal/modelling.py](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab#diff-c3JjL2VkZ2VzX2NhbC9tb2RlbGxpbmcucHk=) | `94.72% <83.33%> (-0.30%)` | :arrow_down: | | [src/edges\_cal/reflection\_coefficient.py](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab#diff-c3JjL2VkZ2VzX2NhbC9yZWZsZWN0aW9uX2NvZWZmaWNpZW50LnB5) | `96.87% <100.00%> (-2.35%)` | :arrow_down: | | [src/edges\_cal/spectra.py](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab#diff-c3JjL2VkZ2VzX2NhbC9zcGVjdHJhLnB5) | `80.95% <100.00%> (ø)` | | | [src/edges\_cal/xrfi.py](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab#diff-c3JjL2VkZ2VzX2NhbC94cmZpLnB5) | `85.96% <100.00%> (-1.00%)` | :arrow_down: | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/edges-collab/edges-cal/pull/116/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=edges-collab)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.