Stone-Soup icon indicating copy to clipboard operation
Stone-Soup copied to clipboard

Add Accumulated State Densities with out of sequence measurements

Open jogehl opened this issue 4 years ago • 4 comments

This branch adds a simple implementation of Accumulated State Densities. They can be used not only for in sequence, but also for the out of sequence case.

jogehl avatar Sep 29 '20 18:09 jogehl

@jogehl Would it be possible to provide an example of ASD in action as part of the documentation. Maybe just adapting one of the existing tutorials or examples, but putting the detections out of sequence?

If you haven't already, you'll need to install the documentation dependencies for Stone Soup which you can find in the README (or can be done easily from the base directory with python -m pip install -e .[dev]). Then from base of the repo run python -m sphinx docs/source docs/build. This will build the documentation and execute the examples, outputting documentation into the docs/build/ directory.

sdhiscocks avatar Oct 07 '20 09:10 sdhiscocks

Codecov Report

Merging #309 into master will increase coverage by 0.31%. The diff coverage is 91.50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #309      +/-   ##
==========================================
+ Coverage   93.61%   93.93%   +0.31%     
==========================================
  Files         109      126      +17     
  Lines        3665     5307    +1642     
  Branches      528      753     +225     
==========================================
+ Hits         3431     4985    +1554     
- Misses        153      227      +74     
- Partials       81       95      +14     
Flag Coverage Δ
#integration 71.96% <69.47%> (?)
#unittests 90.37% <83.39%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
stonesoup/metricgenerator/tracktotruthmetrics.py 93.13% <ø> (+1.96%) :arrow_up:
stonesoup/sensor/radar/radar.py 95.42% <ø> (-0.41%) :arrow_down:
stonesoup/sensor/sensor.py 100.00% <ø> (ø)
stonesoup/serialise.py 99.29% <ø> (+2.66%) :arrow_up:
stonesoup/simulator/platform.py 100.00% <ø> (ø)
stonesoup/simulator/simple.py 97.47% <ø> (-2.53%) :arrow_down:
stonesoup/simulator/transition.py 95.39% <ø> (ø)
stonesoup/smoother/base.py 100.00% <ø> (ø)
stonesoup/smoother/lineargaussian.py 82.92% <ø> (-9.94%) :arrow_down:
stonesoup/tracker/pointprocess.py 98.36% <ø> (ø)
... and 132 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 ff32002...d740d39. Read the comment docs.

codecov[bot] avatar Oct 12 '20 14:10 codecov[bot]

Prior to merging, could commit messages be reworded to give context please?

oharrald-Dstl avatar Nov 04 '21 10:11 oharrald-Dstl

Apologies on the delay reviewing and merging this @jogehl. I've opened #734 with minor changes and updating code to work with latest version of the main code base.

sdhiscocks avatar Oct 07 '22 12:10 sdhiscocks

Resolved in #734

sdhiscocks avatar Feb 20 '23 15:02 sdhiscocks