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

Make LuST compatible with SUMO 1.0.1 #6

Closed

Conversation

kenog
Copy link

@kenog kenog commented Dec 19, 2018

In order to use LuST with SUMO 1.0.1, the file names in <route-files>
and <additional-files> in *.sumocfg need to be separated by commas.

I also tested this patch with SUMO 0.32.0 and it works as well.

In order to use LuST with SUMO 1.0.1, the file names in <route-files>
and <additional-files> in *.sumocfg need to be separated by commas.
@lcodeca
Copy link
Owner

lcodeca commented Dec 19, 2018

I really appreciate the fact that you spent the time to make this pull request, but we have a different issue going on here, and it is not the comma in the files.

The mobility models in sumo have changed over time, and when I tried to move it to sumo 0.27 we have found bugs and quirks in the traffic demand, so I had to roll back to 0.26. This scenario has been validated with sumo 0.26, and I don't have access to the validation data anymore. Even if we make it SUMO 1.1.0 compatible, it's not going to be validated.

In order to port it to the current version of sumo, the network must be re-converted, the TLS definitions must be updated, the realism of the vehicles' behavior has to be re-validated, the DUE mobilities have to be recomputed from scratch and finally, the outcome must be validated with the real data that I don't have anymore.

If you don't care about a validated mobility but you want to use all the latest features of sumo, I suggest you to try https://github.com/lcodeca/MoSTScenario.

@kenog
Copy link
Author

kenog commented Dec 21, 2018

Oh wow ok. I just noticed sumo complaining about not being able to read the route files and had a look at MoST. There I found the commas and since the sumo accepted the files after adding them, I naively assumed that this was it.
However, thank you very much for the detailed explanation. Fortunately were able to use SUMO 0.26 in our simulation with only minor adjustments.

@lcodeca
Copy link
Owner

lcodeca commented Dec 21, 2018

No problem, I'm always happy to help!

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.

None yet

2 participants