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

Lapper and positioner don't agree #144

Open
redfast00 opened this issue Apr 23, 2024 · 1 comment
Open

Lapper and positioner don't agree #144

redfast00 opened this issue Apr 23, 2024 · 1 comment

Comments

@redfast00
Copy link
Member

Currently, the lapper (robust lapper) and posititioner (nostradamus) don't have the same result at the same moment: the lapper always runs on all data, while the positioner is incremental (only calculates on new data). The lapper has to calculate a lot more, and thus takes about 5 seconds per lap calculation, so can't be run as fast as the positioner (which runs every 500ms).

Because they don't agree on position, the positioner sometimes says the lap has 100% progress, but no new lap is added until the lap progress is 25% (of the next lap). This looks weird in the visualisation, so we can't show them at the same time.

@redfast00
Copy link
Member Author

A possible solution could be to integrate the lapper and positioner into one class (and make it incremental), but then we need to think about recovering from disconnected stations; position data does not really matter, but lap count does. Some conditions we need to think about:

  • one station is disconnected for ~5 minutes, then is reconnected, so all 'old' detections are added to the DB
  • all stations are disconnected for ~5 minutes
  • all but one station are disconnected for ~5 minutes

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

No branches or pull requests

1 participant