Closed mkolopanis closed 3 years ago
Merging #347 (a43e80e) into main (13bb52b) will decrease coverage by
0.00%
. The diff coverage isn/a
.
@@ Coverage Diff @@
## main #347 +/- ##
==========================================
- Coverage 99.02% 99.02% -0.01%
==========================================
Files 13 13
Lines 1945 1943 -2
==========================================
- Hits 1926 1924 -2
Misses 19 19
Impacted Files | Coverage Δ | |
---|---|---|
pyuvsim/uvsim.py | 98.45% <ø> (-0.01%) |
:arrow_down: |
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 13bb52b...a43e80e. Read the comment docs.
@bhazelton I think this will require admin to merge because I've removed covered lines, but I don't think it is reasonable to serialize a uvtask to once a second.
I don't think you do -- the test that's failing is not a required test for exactly this reason.
Oh okay thanks! Always get scared at red x
On Tue, Apr 6, 2021, 16:33 Bryna Hazelton @.***> wrote:
I don't think you do -- the test that's failing is not a required tea for exactly this reason.
— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/RadioAstronomySoftwareGroup/pyuvsim/pull/347#issuecomment-814498963, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACB7BLINGDQFOBNXYAVPWO3THOK23ANCNFSM42PUDA2Q .
remove sleep introduced in last PR. Turns out it serializes the execution of the task loop, Ran reference sim 1.1 before and after this change and the before is very slow, but after completes in ~.03 minutes same as before last PR
Types of changes
Checklist:
For all pull requests: