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

Ability to select multiple weight limit signs #5803

Open
vfosnar opened this issue Aug 16, 2024 · 12 comments
Open

Ability to select multiple weight limit signs #5803

vfosnar opened this issue Aug 16, 2024 · 12 comments
Labels
enhancement needs PR accepted suggestion, but only if a contributor implements it

Comments

@vfosnar
Copy link

vfosnar commented Aug 16, 2024

Use case
At least here in Czech Republic it's common to have 2 signs describing weight limit for different types of vehicles.

Proposed Solution
Add option to add more than one weight limit sign

@westnordost
Copy link
Member

Can you post a few pictures?

@vfosnar
Copy link
Author

vfosnar commented Aug 17, 2024

It's usually these two on a single pole
B13
B14

@westnordost
Copy link
Member

I meant photos, but okay.

To gauge whether it is worth the time, effort and added UI complexity to implement this as opposed to let users leave a note (with photo) instead, we need to know how often this actually occurs, we need some numbers. How many notes are created for the weight limit quest? On how many bridges we have multiple different maxweight restrictions?

@westnordost westnordost added the feedback required more info is needed, issue will be likely closed if it is not provided label Aug 17, 2024
@vfosnar
Copy link
Author

vfosnar commented Aug 17, 2024

I meant photos, but okay.

got that, just don't have a photo and couldn't find any on the internet

How many notes are created for the weight limit quest?

not sure but overall weight limits are heavily underreported here in ČR

I can extract some statistics when I get home

@kmpoppe

This comment was marked as resolved.

@westnordost

This comment was marked as resolved.

@vfosnar
Copy link
Author

vfosnar commented Aug 23, 2024

20240823_193720_lmc_8 4

@vfosnar

This comment was marked as off-topic.

@vfosnar vfosnar closed this as completed Aug 23, 2024
@vfosnar vfosnar reopened this Aug 23, 2024
@vfosnar
Copy link
Author

vfosnar commented Aug 23, 2024

Sorry my phone froze after loading an overpass query and ghost pressed the 'close with comment' button :D

for Czechia:

  • 285 have both maxaxleload and maxweight
  • 312 have maxaxleload
  • 8806 have maxweight

so maxaxleload is in ~91% times tagged in this combination and maxweight in ~3%

for the record I didn't see this combination tagged wrongly as 'maxweight only' yet, but I also don't actively check already mapped access restrictions

@westnordost
Copy link
Member

westnordost commented Aug 23, 2024

Hm, well. That doesn't sound worth it to implement (, so I won't).

But I would merge a PR that implements this.

@westnordost westnordost added enhancement and removed feedback required more info is needed, issue will be likely closed if it is not provided labels Aug 23, 2024
@mnalis mnalis added the needs PR accepted suggestion, but only if a contributor implements it label Aug 23, 2024
@RubenKelevra

This comment was marked as resolved.

@westnordost

This comment was marked as resolved.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement needs PR accepted suggestion, but only if a contributor implements it
Projects
None yet
Development

No branches or pull requests

5 participants