search

osmberlin / osm-traffic-sign-tool

A proof of concept tool to help you find the right `traffic_sign=*` tag as well as recommended tags for the road that it applies to.
https://trafficsigns.osm-verkehrswende.org/
GNU Affero General Public License v3.0
18 stars 5 forks source link

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

Open Tillbtn opened 3 weeks ago

Tillbtn commented 3 weeks ago

Link

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

tordans commented 2 weeks ago

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 …

tordans commented 2 weeks ago

Das funktioniert – wie befürchtet – noch nicht.

https://trafficsigns.osm-verkehrswende.org/DE?signs=DE:264%5B2.2%5D,1020-30 aus https://github.com/osmberlin/osm-traffic-sign-tool/issues/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 commented 2 weeks ago

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 :)

Tillbtn commented 2 weeks ago

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 commented 1 day ago

Danke fürs erneute Fehler-Finden, @Tillbtn

Ich habe gerade diese Logik live geschaltet:

Zusatzzeichen können jetzt einen von zwei Typen haben:

For condition_modifier, the primary condition is removed and only the *:conditional tag stays. Eg. maxspeed:conditional=30 @ (22-06)

Beispiel

For exception_modifier, the primary condition stays but is resolved conditionally. Eg. maxweight=5.5 + maxweight:conditional=none @ (destination)

Beispiel

Morgen werde ich das ganze mehr testen.

Tillbtn commented 21 hours ago

sieht gut aus. Bei ein paar Zusatzschildern funktioniert die Auswertung noch nicht, relevant sind da wahrscheinlich: Linienverkehr, evtl. Einsatzfahrzeuge, LKW (wobei das eigentlich nicht allzu viel Sinn ergibt - für wen würde das Verbot dann überhaupt gelten?) und Traktoren