Skip to content

Add Truck Speed Limit field#520

Merged
tyrasd merged 2 commits intoopenstreetmap:mainfrom
arch0345:hgv-maxspeed
Jul 15, 2022
Merged

Add Truck Speed Limit field#520
tyrasd merged 2 commits intoopenstreetmap:mainfrom
arch0345:hgv-maxspeed

Conversation

@arch0345
Copy link
Copy Markdown
Contributor

@arch0345 arch0345 commented Jul 8, 2022

This PR adds a field for maxspeed:hgv, which I've added to the Motorway, Trunk Road, and Primary Road presets.

@tyrasd tyrasd added the new-field create a new field (see add-field for cases where field from presets is added to new entries) label Jul 11, 2022
@tyrasd
Copy link
Copy Markdown
Member

tyrasd commented Jul 11, 2022

would you like to add this field also to the US+Canada regional version of the primary_link preset? primary_link-US-CA.json

@arch0345
Copy link
Copy Markdown
Contributor Author

would you like to add this field also to the US+Canada regional version of the primary_link preset? primary_link-US-CA.json

Most primary links in the US/Canada only have an advisory speed limit. Currently there are only 2 primary links in the US/Canada with maxspeed:hgv: https://overpass-turbo.eu/s/1k3g

image

@tyrasd
Copy link
Copy Markdown
Member

tyrasd commented Jul 12, 2022

You mean the corresponding appropriate tag for the US/CA would be maxspeed:hgv:advisory? Ok, I guess we can skip it then

@arch0345
Copy link
Copy Markdown
Contributor Author

Whoops, just realized I added it to the Primary Link preset instead of the Primary Road preset.

Combination with `highway=primary_link` is only used 200 times compared to the 12K features tagged with `highway=primary`
@tyrasd tyrasd merged commit 260123a into openstreetmap:main Jul 15, 2022
@arch0345 arch0345 deleted the hgv-maxspeed branch July 15, 2022 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-field create a new field (see add-field for cases where field from presets is added to new entries)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants