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

Drt speed up #43

Merged
merged 27 commits into from
Feb 23, 2024
Merged

Drt speed up #43

merged 27 commits into from
Feb 23, 2024

Conversation

tschlenther
Copy link
Contributor

  • Read all the necessary inforation from the drt input shape file (--drt-area). (The corresponding file needs to be updated in SVN!)
  • remove command line options --drt-modes and --drt-case
  • clean code
  • set up with drt-speed up technically works. In first and last iteration, drt is simulated, not teleported.
  • configuration still is in the code

ToDos (for separate PR):

  • ToDo: make the ASC and the other speedUp parameters accessible from outside (i.e. from config or command line), again.
  • ToDo: potentially get rid of the configuration in the code, generally!?
    --> this will simplify the caliration process by large means

@GregorRyb you branched/started from the main branch. I am not sure whether this should be merged back into there or into some other branch.

@tschlenther
Copy link
Contributor Author

Also ToDo:

The DrtCaseSetup generates and saves drt input files next to the input config. This means, those files are overriden for each new run that is started. Really bad, in case we change something for one of the runs but not for all, for example the #vehciles for one of the areas.

@tschlenther tschlenther merged commit a473a93 into main Feb 23, 2024
4 of 5 checks passed
@tschlenther tschlenther deleted the drtSpeedUp branch February 23, 2024 10:48
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