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

DE:250,1020-12: vehicle=no statt vehicle=destination wird vorgeschlagen #58

Open
Tillbtn opened this issue Nov 2, 2024 · 4 comments
Open
Assignees

Comments

@Tillbtn
Copy link

Tillbtn commented Nov 2, 2024

Link

https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:250,1020-12

@Tillbtn Tillbtn changed the title DE:250,1020-12 : vehicle=no statt vehicle=destination wird vorgeschlagen DE:250,1020-12: vehicle=no statt vehicle=destination wird vorgeschlagen Nov 2, 2024
@tordans tordans closed this as completed in 0e8e808 Nov 8, 2024
@tordans
Copy link
Contributor

tordans commented Nov 8, 2024

Thank you for reporting! This is now fixed by adding the 'destination' as modifier for conditional and access restrictions.
https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:250,1020-12

Ideally, please double check other cases …

  • it might be, that other signs need this double tagging
  • it might be, that now new combinations show up that are uninteded

@tordans tordans self-assigned this Nov 8, 2024
@tordans
Copy link
Contributor

tordans commented Nov 8, 2024

Das funktioniert – wie befürchtet – noch nicht.

https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:264%5B2.2%5D,1020-30 aus #45

Hier ist das Tagging jetzt

…
access=destination
maxwidth:conditional=2.2 @ destination

… und der access-Wert hat hier nichts zu suchen.

Ich muss also das Datenschema nochmal ändern. Es darf dann nur einen modifierValue geben, der auf beide cases, den conditional und den access case angewendet wird. Bin nur noch unsicher, ob das immer so ist, oder ob ich eine neue Weiche einbauen muss.

@tordans
Copy link
Contributor

tordans commented Nov 8, 2024

Nächster Versuch…

Ich prüfe jetzt, ob das Haupt-Zeichen ein conditional oder access Zeichen ist und wende dann die Modifikatoren-Werte jew. nur auf das passende an. Zusätzlich muss man aber noch einen top level access in bestimmten Fällen setzen.

@Tillbtn bitte teste nochmal mit …

Ich schließe schonmal, in der Hoffnung, dass es stimmt … und öffne wieder, wenn nicht :)

@tordans tordans closed this as completed Nov 8, 2024
@Tillbtn
Copy link
Author

Tillbtn commented Nov 8, 2024

Ich fürchte, du musst das Issue nochmal öffnen.

Für https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:264%5B2.2%5D,1020-30 und die anderen Beschränkungen, von denen Anlieger ausgenommen sind (Höhe, Gewicht), müsste gelten:

maxwidth=2.2
maxwidth:conditional=none @ destination

gerade wird mit maxwidth:conditional=2.2 @ destination das Gegenteil empfohlen, also eine Beschränkung, die nur für Anlieger gilt.

Außerdem kommt bei Hinzufügen von "Anlieger frei" jetzt immer eine highway=* Empfehlung dazu.

Was mir noch aufgefallen ist: bei https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:250,1022-10 kommt nun ein highway=footway hinzu, das ist wahrscheinlich auch nicht gewollt

@tordans tordans reopened this Nov 8, 2024
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

2 participants