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 in a new TimeTravel mesage to allow the system to change the NUClear clock and ensure that chrono events continue to work #102

Merged
merged 46 commits into from
Apr 17, 2024

Conversation

TrentHouliston
Copy link
Member

@TrentHouliston TrentHouliston commented Feb 26, 2024

This PR adds in a new TimeTravel mesage to allow the system to change the NUClear clock and ensure that chrono events continue to work.

The following time travel types are added:

  • RELATIVE Adjust clock and move all chrono tasks with it
  • ABSOLUTE Adjust clock to target time and leave chrono tasks where they are
  • NEAREST Adjust clock to as close to target as possible without skipping any chrono tasks

…ear clock and ensure that messages continue to work
@TrentHouliston TrentHouliston changed the title Add in a new TimeTravel mesage to allow the system to change the NUClear clock and ensure that messages continue to work Add in a new TimeTravel mesage to allow the system to change the NUClear clock and ensure that chrono events continue to work Feb 26, 2024
tests/individual/TimeTravel.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravel.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravel.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravel.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravel.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravelFrozen.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravelFrozen.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravelFrozen.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravelFrozen.cpp Outdated Show resolved Hide resolved
tests/individual/TimeTravelFrozen.cpp Outdated Show resolved Hide resolved
@Tom0Brien Tom0Brien requested a review from Bidski April 17, 2024 03:21
@TrentHouliston TrentHouliston marked this pull request as ready for review April 17, 2024 03:21
@TrentHouliston TrentHouliston merged commit 98e9dea into main Apr 17, 2024
13 checks passed
@TrentHouliston TrentHouliston deleted the houliston/timetravel branch April 17, 2024 04:01
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.

2 participants