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

Initial PUT request to /sim does not set timeSlotDuration #22

Open
remcollier opened this issue Jul 24, 2021 · 3 comments
Open

Initial PUT request to /sim does not set timeSlotDuration #22

remcollier opened this issue Jul 24, 2021 · 3 comments

Comments

@remcollier
Copy link

The initial PUT to /sim does not set the timeSlotDuration field in SimulationEngine.java. There is a TODO comment associated with the declaration of the field. To change the duration, we have had to download the server and change the default value manually.

@vcharpenay
Copy link
Contributor

In data/sim.ttl (the example file to start the simulation) as well as on the HTML tutorial, there's the following comment:

:timeslotDuration 60000 ; # in ms (corresponds to simulated time, not real execution time)

What you'd like to configure is the real execution time of 100ms, right?

@vcharpenay
Copy link
Contributor

I first thought timeslot duration (in real exection time) would rather go in the .properties file used to configure the server at start time. (I can implement that rather quickly.)

What would be your use case for setting this value via /sim instead of via a .properties file?

@remcollier
Copy link
Author

so that is probably another one to chalk up to not reading the manual.

a properties file would be perfect for configuring the real execution time.

We are using 2000ms at the moment so the agents have time to respond within a smaller number of iterations, but may go higher (maybe 10000ms) when recording demos to give us time to voice over what is happening...

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

2 participants