HERA-Team / hera_cal

Library for HERA data reduction, including redundant calibration, absolute calibration, and LST-binning.
MIT License
13 stars 8 forks source link

Fix some errors related to recent pyuvdata changes #908

Closed bhazelton closed 1 year ago

bhazelton commented 1 year ago

Fixes a couple of bugs with the newest pyuvdata. There are still some test errors that I don't think are related to pyuvdata changes.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 96.95% and project coverage change: -0.03% :warning:

Comparison is base (829c2de) 97.20% compared to head (7fe298d) 97.17%. Report is 234 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #908 +/- ## ========================================== - Coverage 97.20% 97.17% -0.03% ========================================== Files 21 23 +2 Lines 9455 10282 +827 ========================================== + Hits 9191 9992 +801 - Misses 264 290 +26 ``` | [Files Changed](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team) | Coverage Δ | | |---|---|---| | [hera\_cal/\_cli\_tools.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvX2NsaV90b29scy5weQ==) | `97.56% <ø> (ø)` | | | [hera\_cal/abscal.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvYWJzY2FsLnB5) | `96.99% <ø> (ø)` | | | [hera\_cal/vis\_clean.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvdmlzX2NsZWFuLnB5) | `97.77% <ø> (ø)` | | | [hera\_cal/utils.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvdXRpbHMucHk=) | `95.58% <93.24%> (-0.67%)` | :arrow_down: | | [hera\_cal/io.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvaW8ucHk=) | `98.15% <96.36%> (-0.21%)` | :arrow_down: | | [hera\_cal/lstbin.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvbHN0YmluLnB5) | `96.90% <97.22%> (+0.23%)` | :arrow_up: | | [hera\_cal/lstbin\_simple.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvbHN0YmluX3NpbXBsZS5weQ==) | `97.65% <97.65%> (ø)` | | | [hera\_cal/datacontainer.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvZGF0YWNvbnRhaW5lci5weQ==) | `98.69% <100.00%> (+0.32%)` | :arrow_up: | | [hera\_cal/delay\_filter.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvZGVsYXlfZmlsdGVyLnB5) | `100.00% <100.00%> (ø)` | | | [hera\_cal/frf.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvZnJmLnB5) | `97.53% <100.00%> (+0.03%)` | :arrow_up: | | ... and [2 more](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/908?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team) | |

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