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

Add presets for highway=path-based cycle-footway tagging #1384

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

olafkryus
Copy link
Contributor

@olafkryus olafkryus commented Nov 6, 2024

Description, Motivation & Context

Some of the regions are opted out of the cycle-footway preset currently available in iD, since the preset insists on using the highway=cycleway tag, but those communities accepted the use of highway=path tag as their consensus on tagging the feature. Yet no alternative preset is available, therefore in these regions, those cycle-footways are displayed in iD in same way as regular path, with no way to apply any preset, neither cycleway nor path-based one. This PR aims to fill the gap.

Related issues

Copy link

github-actions bot commented Nov 6, 2024

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

@matkoniecz
Copy link
Contributor

Partially resolves

will it cause github to close linked issue? maybe put "resolves (partially)" ?

@tordans

This comment was marked as outdated.

@tordans

This comment was marked as outdated.

@olafkryus

This comment was marked as resolved.

@olafkryus
Copy link
Contributor Author

olafkryus commented Dec 14, 2024

I've also now added Norway to the inclusion and exclusion country lists in the presets concerned, as per #1193, since those country lists should be identical between those presets, and since #1193 ( which seems to have been "forgotten" for some reason? 🤷‍♂️ ) is covering roughly the same issue. So if this PR is deemed to be good to go, I guess #1193 should be merged first, in order to have correct contributor attributions in main repo.

@tordans
Copy link
Collaborator

tordans commented Dec 19, 2024

@olafkryus please use the test template to add some test cases to this PR https://github.com/openstreetmap/id-tagging-schema/blob/main/.github/PULL_REQUEST_TEMPLATE.md?plain=1#L24-L70

I also commented on #1193 (comment) and you might have an opinion on this.

TBH I am not sure that adding the regional preset changes to this PR will lead to this one being merged sooner. It might be better to keep those things separated.

@tordans
Copy link
Collaborator

tordans commented Dec 19, 2024

@olafkryus

TBH I am not sure that adding the regional preset changes to this PR will lead to this one being merged sooner. It might be better to keep those things separated.

Given how the conversation went in #1193 I would like to tune back this PR to only include the added preset (and maybe the missing "de" https://github.com/openstreetmap/id-tagging-schema/pull/1384/files#diff-3b8f616654b5d2845f63eb05ce32b27741d3945ad0a71e55033de12a656d8b56R7)

I would consider this ready for merge then. But we still should check the test checklist first.

@olafkryus
Copy link
Contributor Author

Test-Documentation

Preview links & Sidebar Screenshots

Icons are copied, and fields are directly re-used from already existing related highway=cycleway-based tagging presets.

Bicycle & Foot Path

https://pr-1384--ideditor-presets-preview.netlify.app/id/dist/#background=Geoportal2-PL-aerial_image_WMTS&disable_features=boundaries&id=w991414746&locale=en&map=20.22/50.26270/19.00850
obraz

Bicycle & Foot Crossing

https://pr-1384--ideditor-presets-preview.netlify.app/id/dist/#background=Geoportal2-PL-aerial_image_WMTS&disable_features=boundaries&id=w699046787&locale=en&map=20.22/50.26270/19.00850
obraz

Search

obraz
obraz

Info-i

Bicycle & Foot Path

obraz

Bicycle & Foot Crossing

obraz

Wording

New presets directly re-use all wording from already existing related highway=cycleway-based tagging presets.

@olafkryus
Copy link
Contributor Author

@tordans If I had paid a bit more attention to the description of #1193, I probably would have noticed that it's similarly-looking solution is actually an unexpected method of trying to solve a bit different problem. 🫥 Whether cycleway is implying bicycle designation is quite a good point, but certainly for another issue. (on the bright side, at least that moved forward a bit 😉 ). For now, it's Norway-out from the country lists here.

Test Documentation added.

@tordans
Copy link
Collaborator

tordans commented Dec 20, 2024

It's great to have this in place, now. Looking at it on the map makes me hope we will find a good solution for openstreetmap/iD#10256 which would resolve the rendering issue of those shared path lines. — Maybe you have an idea over there @olafkryus

@tordans
Copy link
Collaborator

tordans commented Dec 20, 2024

Documentation:

The info-i part is a bit unfortunate but I don't see any way to reference a wiki page that will show something with a more to the point meaning. It is all very generic hw=path and not what we would want to see here like "shared bike and pedestrian path/crossing, segregated or not".

When I switch from a shared path to a cycle path and back via presets the "segregated=*" value gets lost. But I think that is OK for this use case.

Search looks good image

@tordans tordans merged commit bc74fb5 into openstreetmap:main Dec 20, 2024
5 checks passed
@1ec5
Copy link
Contributor

1ec5 commented Dec 20, 2024

The info-i part is a bit unfortunate but I don't see any way to reference a wiki page that will show something with a more to the point meaning. It is all very generic hw=path and not what we would want to see here like "shared bike and pedestrian path/crossing, segregated or not".

That’s a known issue with the tagging scheme overall. I’m not sure we can reasonably expect a single brief description that covers the tag’s multiple personalities. There are too many relevant forum threads to list here, but here are a few off the top of my head:

@tordans
Copy link
Collaborator

tordans commented Dec 20, 2024

@1ec5 I was thinking along the lines of …

  • find a way to model a wiki data item for a tag combination and use that as reference for a preset –> AFAIK that is not part of the wiki data item setup at this point
  • have an optional hard coded info-i text here in the code –> this could get messy and would move more responsibility here which I don't think is worth the effort at this point

The reference cannot help here due the lack of a specific tag…

@1ec5
Copy link
Contributor

1ec5 commented Dec 20, 2024

The data item’s description already has a disclaimer in English and possibly some other languages that would be affected by this preset. Maybe the disclaimer could say that the tag is not just unspecific but overloaded with multiple meanings, so the user should read the full documentation for details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cycle & Foot Path preset missing from search results in France New preset: Cycle & Foot Path for Europe
4 participants