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

remove ptCounts #2853

Merged
merged 2 commits into from
Oct 12, 2023
Merged

remove ptCounts #2853

merged 2 commits into from
Oct 12, 2023

Conversation

mrieser
Copy link
Contributor

@mrieser 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.

Closes #2845

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 enabled auto-merge October 12, 2023 15:49
@mrieser mrieser merged commit 4f84a77 into master Oct 12, 2023
48 checks passed
@mrieser mrieser deleted the 2845-remove-ptcounts branch October 12, 2023 15:58
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.

Remove ptCounts
1 participant