Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Progress Trackers Manually: Option C #888

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

gawebb-dstl
Copy link
Contributor

All current trackers require a detection feeder. It can be useful to progress a tracker manually without a detection feeder (see Multi-Sensor Fusion: Covariance Intersection Using Tracks as Measurements example).

All current trackers and start their __next__ function with:

time, detections = next(self.detector_iter)
# Some tracking logic

I've moved this line to the new Tracker baseclass and added an update_tracker function which takes the time and detections from the detection feeder as an input. With this being a separate function, you can access it directly and bypass the detection feeder which may be easier in instances.

This also fixes issue #883

This is my third iteration of trying to get this right. See #742 and #779 for previous implementations.

Removed mention of DummyDetector in Track2Track_Fusion_Example.py as it wasn't needed
@gawebb-dstl gawebb-dstl requested a review from a team as a code owner November 7, 2023 17:34
@gawebb-dstl gawebb-dstl requested review from nperree-dstl, hpritchett-dstl and sdhiscocks and removed request for a team and hpritchett-dstl November 7, 2023 17:34
@gawebb-dstl
Copy link
Contributor Author

The changes to docs/examples/Track2Track_Fusion_Example.py can probably be removed now #882 has been merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants