HERA-Team / hera_sim

Simple simulation code for HERA-like redundant interferometric arrays
Other
16 stars 8 forks source link

fix: make uvbeam optional in MutualCoupling init #250

Closed r-pascua closed 1 year ago

r-pascua commented 1 year ago

It isn't necessary to provide an argument for the uvbeam parameter when initializing the MutualCoupling class, but this parameter was required prior to this update. This PR fixes that issue.

codecov[bot] commented 1 year ago

Codecov Report

Base: 96.55% // Head: 96.55% // No change to project coverage :thumbsup:

Coverage data is based on head (d4bb723) compared to base (68db6d9). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #250 +/- ## ======================================= Coverage 96.55% 96.55% ======================================= Files 24 24 Lines 3046 3046 ======================================= Hits 2941 2941 Misses 105 105 ``` | [Impacted Files](https://codecov.io/gh/HERA-Team/hera_sim/pull/250?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team) | Coverage Δ | | |---|---|---| | [hera\_sim/sigchain.py](https://codecov.io/gh/HERA-Team/hera_sim/pull/250?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9zaW0vc2lnY2hhaW4ucHk=) | `98.48% <ø> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team)

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