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

Complain about "Unspecific primary tag value" (like barrier=yes) in validator #9209

Open
matkoniecz opened this issue Jul 22, 2022 · 6 comments
Labels
validation An issue with the validation or Q/A code

Comments

@matkoniecz
Copy link
Contributor

Description

Offer to replace it with more specific feature

Offer to split it into parts (does not apply to nodes and amenity=school barrier=yes taggings)

From openstreetmap/id-tagging-schema#531

Screenshots

No response

@matkoniecz
Copy link
Contributor Author

BTW, the same can be done with say https://taginfo.openstreetmap.org/tags/traffic_calming=yes

@tyrasd tyrasd added the validation An issue with the validation or Q/A code label Jul 22, 2022
@tyrasd tyrasd changed the title barrier=yes and complain about "Unspecific barrier value" in validator Complain about "Unspecific primary tag value" (like barrier=yes) in validator Jul 22, 2022
@tyrasd
Copy link
Member

tyrasd commented Jul 22, 2022

Here's a quick list of all primary tag keys with their usage numbers of their yes value (only showing tags with usage numbers > 2k, and excluding building=yes for obvious reasons):

Marked with ✅ are tags where the id-tagging-schema already includes a dedicated hidden <primary tag> (Unspecified Type) preset.

I guess what the validator could do is the following: If the primary tag(s) of an object all have the value yes (adding an exception for building=yes), show a warning/information about the lack of detail. That way a feature tagged as barrier=city_wall + historic=yes (or amenity=school + barrier=yes) would not be flagged.

Footnotes

  1. I'd assume that these tags might largely not be used as a primary tag (with the value yes), but rather as properties of other primary tags 2

@tyrasd
Copy link
Member

tyrasd commented Jul 22, 2022

Offer to split it into parts

What exactly do you mean by splitting into parts here? And how would it help to solve the issue?

@matkoniecz
Copy link
Contributor Author

What exactly do you mean by splitting into parts here? And how would it help to solve the issue?

It is possible that barrier=yes is partially representing barrier=fence and partially barrier=wall

In such case it should be split in pars before being tagged with more specific value.

Though maybe it can be assumed that user is capable of doing it with a general tools?

@matkoniecz
Copy link
Contributor Author

@tyrasd

Would it make sense to request adding "Unspecific Foobar" on iD tagging scheme issue tracker for this list from #9209 (comment) ?

@tyrasd
Copy link
Member

tyrasd commented Jul 22, 2022

I've just added support for the remaining common ones (man_made, healthcare, traffic_calming, barrier) in addition to the previously existing ones (shop, office). I think the rest is reasonably uncommon (or used as an non-primary tag / attribute) to not need a dedicated preset IMHO.

I think it's sufficient to just implement the validator to show a hint about all of these (and the remaining primary tags like aerialway=yes, etc. as well).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
validation An issue with the validation or Q/A code
Projects
None yet
Development

No branches or pull requests

2 participants