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.

Sign up for GitHub

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

Jump to bottom

Hide clock and sim_time from user #193

Merged

kilianp14 merged 2 commits into main from hide_sim_time

Mar 20, 2024

Contributor

kilianp14 commented Mar 15, 2024

Removed Clock from public vessim api
Controllers now get the current datetime at every step instead of the simulation time, thus reducing complexity
(closes Hide the clock and sim-time from controllers #188)


          hide sim_time from user

d9af89f

kilianp14 requested review from birnbaum and marvin-steinke

March 15, 2024 11:20

kilianp14 linked an issue

that may be closed by this pull request

Hide the clock and sim-time from controllers #188

Closed

marvin-steinke approved these changes

View reviewed changes

birnbaum approved these changes

View reviewed changes

Collaborator

birnbaum left a comment

great job!

kilianp14 changed the base branch from notebook_integration_tests to main

March 20, 2024 12:04


          trigger build workflow

49c7722

kilianp14 merged commit f9efbdb into main

4 checks passed

kilianp14 deleted the hide_sim_time branch

March 20, 2024 12:12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet