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

Crossing and Cycleway Refactoring Notes #1410

Open
tordans opened this issue Dec 19, 2024 · 0 comments
Open

Crossing and Cycleway Refactoring Notes #1410

tordans opened this issue Dec 19, 2024 · 0 comments

Comments

@tordans
Copy link
Collaborator

tordans commented Dec 19, 2024

While working on #1201 – which is now merged (but not released at this time) – I wrote down a few follow ups that I want to look into as soon as I find the time.

I did not work on them in #1201 to keep the scope on a refactoring and cleanup of what we have.

They are not all thinks that we should to … but thinks I wanted to look into. To give them a bit more visibility I move them from the PR description here so we can reference when things are being worked on …


segregated for cycleways


bicycle_foot versions


crossing:signal field


kerb:left/kerb:right field


traffic_island preset


crossing=informal preset

We have presets for some of the ways, but not for the crossings. However, SC does use the tag.

Notes

  • Make it unsearchable (at first)
  • Don't add it to the translated field-list in order to keep this focussed on the tree main types

crossing=no preset

Notes

  • Make it unsearchable (at first)
  • Don't add it to the translated field-list in order to keep this focussed on the tree main types

Research how common values are

Research if traffic_calming=table is common with crossing=traffic_signals?
And is crossing:island=yes common with crossing=traffic_signals?
If not, we could remove those from the presets for node hw=crossing + crossing=traffic_signals.
Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=n7242739087&locale=en&map=20.00/52.33904/13.65936


infrastructure on crossing presets?

Do we want the infrastructure preset on crossings as moreFields? There are cases like https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=w1009387182&locale=en&map=20.00/52.47153/13.46059 where the crossing is on the bridge. => We can add it to the new @templates/geometry_way_more. "bridge" is used 3k times for "crossing=*" https://taginfo.openstreetmap.org/keys/crossing#combinations


name on crossing presets?

Do we want to give name more visibility on hw=path+path=crossing because path sometimes have names? Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=w947039863&locale=en&map=20.00/52.43023/13.44348
The name is a general field which always shows when present, it does show below the explicit fields, though, see https://pr-1201--ideditor-presets-preview.netlify.app/id/dist/#background=Bing&disable_features=boundaries&id=w947039863&locale=en&map=20.00/52.43023/13.44348 at the end of the list


width on crossing presets?

  • Do we want to add width and width:source as moreTags?
  • For the hw=path we might also add bicycle:width and footway:width as moreTags

bicycle:oneway

Is there a way to add bicycle:oneway=* as kind of synonym for oneway=* on hw=path|cycleway? Both are used but only one is visible as field.


zebra

Add "addTags" for crossing=zebra to add marking zebra. Maybe also crossing_ref=zebra? Testcase https://ideditor.netlify.app/#background=Bing&disable_features=boundaries&id=n9446412638&locale=en&map=20.00/52.32519/13.62391

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

No branches or pull requests

1 participant