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

Add option to bind mobility model ticks to system clock #555

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

Conversation

awlane
Copy link
Collaborator

@awlane awlane commented Nov 22, 2024

Adds parameters 'use_timed_model_mob' and 'timed_model_mob_tick' to mobility model code. When enabled, these will cause mobility ticks to only occur every 'timed_model_mob_tick' seconds as long as the expected number of ticks for this time interval has occurred.

The current behavior does provide a 0.5s sleep time without the plot but does not prevent error from accumulating over time- over 30s, we can see a 100ms divergence from the expected timing of a tick develop when this sleep time is included. This likely varies from system to system, but this in turn means that reproducibility between systems is fraught. This also allows the end user to control this sleep interval and visualize the behavior as it occurs without the plot more easily.

Adds parameters 'use_timed_model_mob' and 'timed_model_mob_tick'
to mobility model code. When enabled, these will cause mobility
ticks to only occur every 'timed_model_mob_tick' seconds as long as
the expected number of ticks for this time interval has occurred.
@awlane awlane changed the title Add option to bind mobility model ticks to real time clock. Add option to bind mobility model ticks to system clock. Nov 22, 2024
@awlane awlane changed the title Add option to bind mobility model ticks to system clock. Add option to bind mobility model ticks to system clock Nov 22, 2024
@awlane awlane marked this pull request as ready for review November 22, 2024 22:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant