-
Notifications
You must be signed in to change notification settings - Fork 1
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
Speed KPI - more than 24 hours, missing links & data structure #24
Comments
@syhwawa, two issues here:
|
Hey @divyasharma-arup, I've done some analysis on the Paris East speed outputs.
In the MATSim configuration, we set
For the TE model, around 0.2% of agents had activities extending beyond 24 hours:
TE results look more reasonable since it actually captured the overnight activities mainly on delivery and depot. It seems to me that the outcomes heavily depend on the approach to population synthesis and the calibration of the simulation.
From the Paris East results, I've observed findings similar to what Val noted for TE. The I'm not sure if I fully understand the current method used. One approach could be to modify the table after the pivot operation, to identify and reinsert rows that might have been excluded due to all their speeds being 0. The |
The speed KPI has a number of issues that I found comparing against Elara outputs for TE.
For reference:
Data structure should be improved by ordering columns from 0-23 (currently 0,1,10,11,12 etc.)
Speed table for TE extends to hour 31!
If speed of link is 0 across all time periods (which presumably means no traffic) then the link is not included in
kpi-speed.csv
. We should still include these links as this is a parameter and provides useful context.There are 18,473 links (see below for an extract) that are included in the gelato output but not found in elara logs. Most of these seem to be links between the same nodes which are written out as 0 across all time periods in gelato. But then there are some exceptions to this list which appear to be different nodes and have speeds associated with them.
The text was updated successfully, but these errors were encountered: