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.

Sign up for GitHub

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

Jump to bottom

Remove ptCounts #2845

Closed
mrieser opened this issue Oct 12, 2023 · 0 comments · Fixed by #2853
Closed

Remove ptCounts #2845

mrieser opened this issue Oct 12, 2023 · 0 comments · Fixed by #2853
Assignees
@mrieser
Labels
code sprint Possible issue(s) for the MATSim code sprint

Comments

@mrieser
Copy link
Contributor

mrieser commented Oct 12, 2023

ptCounts were introduced 10+ years ago, but only used in a single project, IIRC.
With the new Counts-Format (#2780, #2840) the separate file format is no longer needed.
As the feature is not really used, we should remove it instead of refactoring it to the new counts.
@mrieser mrieser added the code sprint Possible issue(s) for the MATSim code sprint label Oct 12, 2023
@mrieser mrieser self-assigned this Oct 12, 2023
@kainagel kainagel moved this to In Progress in CodeSprint 2023 Oct 12, 2023
mrieser added a commit that referenced this issue Oct 12, 2023
@mrieser
remove ptCounts
b85edf1 
They haven't been used in the last 10 years. The upcoming counts_v2 is flexible enough to also support various transit-related counts/measurements.

Closes #2845
@mrieser mrieser mentioned this issue Oct 12, 2023
Merged
@github-project-automation github-project-automation bot moved this from In Progress to Done in CodeSprint 2023 Oct 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mrieser mrieser

Labels
code sprint Possible issue(s) for the MATSim code sprint
Projects
CodeSprint 2023
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

remove ptCounts matsim-org/matsim-libs
1 participant
@mrieser