-
Notifications
You must be signed in to change notification settings - Fork 453
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
Counts v2 format #2840
Counts v2 format #2840
Conversation
…le. Updated MultiModeCount class to new functionality
Question: If it says "interval = 3600" and "t=3600", do you mean the time period from 0 to 3599, or from 3600 to 7199? I think that it should be the latter. (Otherwise, the second that is mentioned in the "t" entry would not even be part of the interval.) If this is already the case, it would help to have the examples start at "t=0". If this is not the case, then I really really think we should change it. We had lots of problems with the fact that the old format started at 1 and not at 0. @mrieser , I think that you said the same two days ago. |
# Conflicts: # contribs/cadytsIntegration/src/main/java/org/matsim/contrib/cadyts/general/CadytsBuilderImpl.java
This PR updates the counts format to v2 introducing more flexibility in the type of data that can be stored. In summary, the counts format can now store:
Example file:
The new API involves using
createAndAddMeasureLocation
,MeasurementLocation
andMeasurable
(see javadoc on usage) instead of existingCount
andVolume
classes.All changes are supposed to be backwards compatible, since the old
Count
andVolume
classes are still present and functional.They will internally use volume measurements of a one-hour interval for the mode car.
Closes #2780