Closed jmbarr closed 2 years ago
Also fails doc tests. No idea why that is and doesn't appear to be to do with these changes.
Looks like these changes cause the Moving Platform Simulator example to hang.
Merging #677 (ffcaf3c) into main (76c4c14) will increase coverage by
2.43%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## main #677 +/- ##
==========================================
+ Coverage 92.17% 94.60% +2.43%
==========================================
Files 168 168
Lines 8482 8487 +5
Branches 1643 1644 +1
==========================================
+ Hits 7818 8029 +211
+ Misses 523 341 -182
+ Partials 141 117 -24
Flag | Coverage Δ | |
---|---|---|
integration | 68.66% <87.50%> (?) |
|
unittests | 92.17% <100.00%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
Impacted Files | Coverage Δ | |
---|---|---|
stonesoup/functions/__init__.py | 93.95% <100.00%> (ø) |
|
stonesoup/models/measurement/nonlinear.py | 98.94% <100.00%> (-0.01%) |
:arrow_down: |
stonesoup/types/array.py | 85.15% <100.00%> (+0.73%) |
:arrow_up: |
stonesoup/metricgenerator/ospametric.py | 86.39% <0.00%> (+0.59%) |
:arrow_up: |
stonesoup/sensor/radar/radar.py | 95.67% <0.00%> (+0.86%) |
:arrow_up: |
stonesoup/simulator/simple.py | 93.33% <0.00%> (+1.33%) |
:arrow_up: |
stonesoup/initiator/composite.py | 96.87% <0.00%> (+1.56%) |
:arrow_up: |
stonesoup/initiator/simple.py | 95.96% <0.00%> (+1.61%) |
:arrow_up: |
stonesoup/sensor/actionable.py | 94.20% <0.00%> (+2.89%) |
:arrow_up: |
stonesoup/metricgenerator/metrictables.py | 91.22% <0.00%> (+3.50%) |
:arrow_up: |
... and 16 more |
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 76c4c14...ffcaf3c. Read the comment docs.
Does as suggested in #673. Iterating over
StateVectors
now yields aStateVector
. Note that this breaks a few tests (which have been amended), so may also break iterations which previously accommodated the old way.Also fails doc tests. No idea why that is and doesn't appear to be to do with these changes.