Add track stitching class and example. #764
Conversation
Co-authored-by: Scott Florence <[email protected]>
spike-dstl
requested review from
sdhiscocks and
hpritchett-dstl
and removed request for
a team
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #764 +/- ##
==========================================
+ Coverage 94.69% 94.76% +0.07%
==========================================
Files 174 175 +1
Lines 9194 9375 +181
Branches 1413 1474 +61
==========================================
+ Hits 8706 8884 +178
Misses 349 349
- Partials 139 142 +3
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
|
Code coverage could be improved. It looks like the backwards methods aren't tested currently. |
| if metric.startswith("SIAP") and not metric.endswith(" at times")} | ||
| siap_time_based = {metrics.get(metric) for metric in metrics if metric.endswith(' at times')} | ||
|
|
||
| _ = SIAPTableGenerator(siap_averages).compute_metric() |
There was a problem hiding this comment.
Would be good to compare before and after, but I can see issue is that stitched tracks aren't interpolated. Probably should add that as separate component that could be used.
sdhiscocks
requested review from
sdhiscocks and
rcgorman-dstl
and removed request for
hpritchett-dstl and
jswright-dstl
| # %% | ||
| # Track Stitcher Class | ||
| # ^^^^^^^^^^^^^^^^^^^^ | ||
| # The cell below contains the track stitcher class. The functions `forward_predict` and | ||
| # `backward_predict` perform the forward and backward predictions respectively (as noted above). If | ||
| # using fowards and backwards stitching, predictions from both methods are merged together. | ||
| # They calculate which pairs of tracks could possibly be stitched together. The function `stitch` | ||
| # uses `forward_predict` and `backward_predict` to pair and 'stitch' track sections together. | ||
|
|
||
| from stonesoup.stitcher import TrackStitcher |
There was a problem hiding this comment.
you could delete this text as most content is covered in the intro and move the import statement down to applying the stitcher
|
Co-authored-by: Roisín Gorman <[email protected]>
Co-authored-by: Scott Florence [email protected]
Pull request to add track stitching example and 'Stitcher' class to main stone soup repository.