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

Update Array based travel time data #1935

Closed
wants to merge 4 commits into from
Closed

Conversation

luchengqi7
Copy link
Collaborator

As mentioned in the issue #1933, the travel time observer does not return the correct value for the travel time for the unused links. This difference is due to the discrete QSim time step (see QSimFreeSpeedTravelTime vs FreeSpeedTravelTime). This fix incorporate the QSim config in the Travel Time data construction, such that the travel time for the unused link can be calculated more accurately.

In the future, we should also consider fixing the map based travel time data.

The travel time observer does not return the correct value for the travel time for the unused links. This difference is due to the discrete QSim time step. This fix incorporate the QSim config in the Travel Time data construction, such that the travel time for the unused link can be calculated more accurately.
@luchengqi7
Copy link
Collaborator Author

Since a lot of other code is depending on this, it is probably not that easy to just fix it directly.

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