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
385 stars 127 forks source link

Minor Addition and Changes to MultiMeasurementInitiator #835

Closed gawebb-dstl closed 8 months ago

gawebb-dstl commented 11 months ago

NoHistoryMultiMeasurementInitiator was created so track metrics wouldn’t take into account the part of the track that was in ‘holding’. ‘Holding’ tracks aren’t available to the tracker. When a holding track is upgraded to a full/sure track it takes its history with it. When assessing tracks post-simulation with metrics the full history is used which can be misrepresentative.

An option was added to skip measurements that don’t have a ReversibleModel measurement model in MultiMeasurementInitiator

gawebb-dstl commented 11 months ago

The docs are failing to build _docs/demos/VideoProcessing.py ModuleNotFoundError: No module named 'pytube'

I don't think this is related to any of my changes

gawebb-dstl commented 9 months ago

I messed up the rebase so the branch history got messy. Instead I’ve recreated the branch. Hopefully it is a lot simpler now.

codecov[bot] commented 9 months ago

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (6000941) 93.77% compared to head (540b96e) 93.79%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #835 +/- ## ========================================== + Coverage 93.77% 93.79% +0.02% ========================================== Files 190 190 Lines 11655 11662 +7 Branches 2412 2415 +3 ========================================== + Hits 10929 10938 +9 + Misses 497 495 -2 Partials 229 229 ``` | [Flag](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl) | Coverage Δ | | |---|---|---| | [integration](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl) | `68.02% <42.85%> (+0.01%)` | :arrow_up: | | [unittests](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl) | `89.00% <71.42%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#carryforward-flags-in-the-pull-request-comment) to find out more. | [Files](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl) | Coverage Δ | | |---|---|---| | [stonesoup/initiator/simple.py](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL2luaXRpYXRvci9zaW1wbGUucHk=) | `94.65% <71.42%> (-1.32%)` | :arrow_down: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/dstl/Stone-Soup/pull/835/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.