search

dstl / Stone-Soup

A software project to provide the target tracking community with a framework for the development and testing of tracking algorithms.
https://stonesoup.rtfd.io
MIT License
400 stars 131 forks source link

Pre-fetch noise values for sensor measure method #712

Closed sdhiscocks closed 1 year ago

sdhiscocks commented 1 year ago

This gives a performance boost as sampling multiple noise points is much more efficient.

There is possibility of a model using a frozen distribution, but there is issue with this currently doing matrix decomposition for every call, meaning little benefit of doing this.

As an example, currently this reduces calls to rvs method from 11.8s to 0.2s in the ADS-B tracking demo.

codecov[bot] commented 1 year ago

Codecov Report

Merging #712 (1a4f537) into main (c1a7f02) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #712      +/-   ##
==========================================
+ Coverage   94.59%   94.60%   +0.01%     
==========================================
  Files         172      172              
  Lines        8802     8823      +21     
  Branches     1698     1707       +9     
==========================================
+ Hits         8326     8347      +21     
  Misses        344      344              
  Partials      132      132
Flag Coverage Δ
integration 70.14% <48.14%> (-0.07%) :arrow_down:
unittests 90.71% <81.48%> (-0.03%) :arrow_down:

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

Impacted Files Coverage Δ
stonesoup/sensor/passive.py 100.00% <100.00%> (ø)
stonesoup/sensor/radar/radar.py 95.95% <100.00%> (+0.28%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.