Closed bhazelton closed 2 years ago
Merging #390 (43d51b3) into main (2d01e1d) will increase coverage by
0.00%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #390 +/- ##
=======================================
Coverage 99.33% 99.34%
=======================================
Files 13 13
Lines 2094 2124 +30
=======================================
+ Hits 2080 2110 +30
Misses 14 14
Impacted Files | Coverage Δ | |
---|---|---|
pyuvsim/simsetup.py | 99.89% <100.00%> (+<0.01%) |
:arrow_up: |
pyuvsim/uvsim.py | 99.42% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 2d01e1d...43d51b3. Read the comment docs.
@steven-murray This PR addresses an issue you made some time ago. Can you take a look?
@steven-murray I just updated the changelog and docs, can you re-review?
Description
return_beams
parameter to theinitialize_uvdata_from_params
function to have it return the beam list and dict. Defaults to True currently, but will default to False starting in version 1.4.return_catname
parameter to theinitialize_catalog_from_params
function to have it return the catalog name. Defaults to True currently, but will default to False starting in version 1.4.Motivation and Context
closes #215
Types of changes
Checklist:
For all pull requests:
New feature checklist: