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

Cycle_barrier fields #1385

Closed
wants to merge 1 commit into from

Conversation

Zverik
Copy link
Collaborator

@Zverik Zverik commented Nov 13, 2024

Adds commonly used fields for barrier=cycle_barrier. Adding after a request from the Danish Cyclists' Association, with the goal of having those in Every Door eventually.

Right now the cycle_barrier key has no extra fields except access.

Relevant tag usage stats:

  • barrier=cycle_barrier: 132k
  • cycle_barrier=*: 32.5k, of which 31.3k account for double, single, and triple
  • cycle_barrier:installation=*: 22k
  • maxspeed:physical=*: 10k
  • spacing, opening, and overlap: 500-600 each.

Copy link

🍱 You can preview the tagging presets of this pull request here.

@Zverik
Copy link
Collaborator Author

Zverik commented Nov 13, 2024

id_cycle_barrier_fields

Huh, this preview feature is fun!

@k-yle
Copy link
Collaborator

k-yle commented Nov 13, 2024

this is a duplicate of #1232 right? I believe we're still waiting on ideditor/schema-builder#87

@Zverik
Copy link
Collaborator Author

Zverik commented Nov 13, 2024

@k-yle okaaaaay, you are correct, this is almost the full duplicate. Except this one does not require a schema change. Maybe we could do this in two steps, first using the 1-out-of-3 approach used here, and then changing to lists when the schema is changed?

@Zverik Zverik closed this Nov 13, 2024
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.

2 participants