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
406 stars 134 forks source link

Make Orbital classes and functions compatible with State Vectors #752

Closed jmbarr closed 1 year ago

jmbarr commented 1 year ago

Rewrote Orbital class (and associated functions) to deal with StateVectors.

codecov[bot] commented 1 year ago

Codecov Report

Base: 94.67% // Head: 94.81% // Increases project coverage by +0.14% :tada:

Coverage data is based on head (eeb0b56) compared to base (ce13b24). Patch coverage: 95.69% of modified lines in pull request are covered.

:exclamation: Current head eeb0b56 differs from pull request most recent head 1f2e641. Consider uploading reports for the commit 1f2e641 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #752 +/- ## ========================================== + Coverage 94.67% 94.81% +0.14% ========================================== Files 174 175 +1 Lines 9197 9447 +250 Branches 1822 1449 -373 ========================================== + Hits 8707 8957 +250 Misses 350 350 Partials 140 140 ``` | Flag | Coverage Δ | | |---|---|---| | integration | `69.37% <20.69%> (-1.17%)` | :arrow_down: | | unittests | `89.71% <95.69%> (+0.57%)` | :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. | [Impacted Files](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl) | Coverage Δ | | |---|---|---| | [stonesoup/types/orbitalstate.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL3R5cGVzL29yYml0YWxzdGF0ZS5weQ==) | `96.24% <94.21%> (+3.46%)` | :arrow_up: | | [stonesoup/reader/astronomical.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL3JlYWRlci9hc3Ryb25vbWljYWwucHk=) | `95.23% <95.23%> (ø)` | | | [stonesoup/functions/orbital.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL2Z1bmN0aW9ucy9vcmJpdGFsLnB5) | `100.00% <100.00%> (ø)` | | | [stonesoup/types/angle.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL3R5cGVzL2FuZ2xlLnB5) | `94.53% <0.00%> (-0.13%)` | :arrow_down: | | [stonesoup/simulator/simple.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL3NpbXVsYXRvci9zaW1wbGUucHk=) | `98.00% <0.00%> (-0.03%)` | :arrow_down: | | [stonesoup/mixturereducer/gaussianmixture.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL21peHR1cmVyZWR1Y2VyL2dhdXNzaWFubWl4dHVyZS5weQ==) | `97.80% <0.00%> (+0.02%)` | :arrow_up: | | [stonesoup/plotter.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL3Bsb3R0ZXIucHk=) | `90.19% <0.00%> (+0.15%)` | :arrow_up: | | [stonesoup/dataassociator/tracktotrack.py](https://codecov.io/gh/dstl/Stone-Soup/pull/752?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl#diff-c3RvbmVzb3VwL2RhdGFhc3NvY2lhdG9yL3RyYWNrdG90cmFjay5weQ==) | `93.42% <0.00%> (+1.31%)` | :arrow_up: | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=dstl)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

sdhiscocks commented 1 year ago

Ignore test failures: they are unrelated and known issue.