Skip to content
This repository has been archived by the owner on Mar 9, 2023. It is now read-only.

Proposal for a rework of the parking:lane schema #263

Open
SupaplexOSM opened this issue Nov 1, 2022 · 3 comments
Open

Proposal for a rework of the parking:lane schema #263

SupaplexOSM opened this issue Nov 1, 2022 · 3 comments

Comments

@SupaplexOSM
Copy link

Hey! For anyone who did not see it, yet: There is a proposal in the works with the goal of improving and reworking the parking:lane schema: https://wiki.openstreetmap.org/wiki/Proposed_features/street_parking_revision

Since street parking is relevant for A/B Street, I wanted to inform you about this. We appreciate any comments and - when the time comes - help with the process of data migration/transferring existing data into the new schema. We are planning "translation" lists and maybe tools to support this. If anyone can help with this, it would be awesome!

@dabreegster
Copy link
Contributor

Hey, thanks for sending along and sorry for being so slow to take a look at this! The proposal looks great to me. When the time comes, there's a few bits of abstreet code to update:

  • osm2lanes / osm2streets parsing of tags. It'll have to handle both
  • apps/parking_mapper/. It can start writing the new schema, and it probably has to recognize both the old and new when toggling

@tordans
Copy link
Collaborator

tordans commented Nov 15, 2022

@dabreegster FYI, the proposal is now officially in it's RFC state. It would be great if you took the time to look for any inconsistencies that you see, given your experience with processing the road space.

Is it OK if we interpret your last comment as endorsement of the proposal? I am thinking about make those more visible in the proposal to clarify that main data consumers and editors are on board.

On the topic of data migration:

We are experimenting with a tool that would help migrating the tags. More at https://github.com/osmberlin/osm-tag-updater. IMO this looks promising…

@dabreegster
Copy link
Contributor

Sorry for the slow response here. I've skimmed over it again and see no issues. I kind of trust folks in the discussion tab are giving a proper review! Yes, I endorse this proposal and will add support for the new schema when I carve out some time. (Or if anyone wants to add it to osm2lanes, it should be a fairly straightforward set of new tests and code changes!)

I'll move this issue to that repo

@dabreegster dabreegster transferred this issue from a-b-street/abstreet Nov 20, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants