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

Processing: Merge segments before calculation #66

Open
tordans opened this issue Mar 14, 2023 · 1 comment
Open

Processing: Merge segments before calculation #66

tordans opened this issue Mar 14, 2023 · 1 comment
Assignees

Comments

@tordans
Copy link
Contributor

tordans commented Mar 14, 2023

image

Location

This looks like a nice edge case:
The OSM way that the "Missing data" Segment belongs to is https://www.openstreetmap.org/way/1133008268. This one gets split by the driveway.

The left over piece is too small for processing. However, we should in fact merge it first with the line next to it which has the same parking tags. Together they would likely have space for 1 more car…

@SupaplexOSM
Copy link
Contributor

SupaplexOSM commented Mar 14, 2023

Ja, das sollten wir auf jeden Fall tun. Das ist eigentlich kein Edge Case, weil diese Art von Fehler jederzeit passiert, wenn sich Attribute der Straße oder einer Parkstreifenseite ändern, was bei detailliert erfassten Daten oft der Fall ist (etwas weiter südlich z.B. auf der Flughafenstraße gibt es auch viele kurze Segmente nebeneinander, obwohl einige davon zusammengehören, weil mal auf der einen, mal auf der anderen Seite die Parkbeschränkungen wechseln).

Das Pythonscript "dissolved" einfach gleich zu Beginn alle Parksegmente mit gleichen Attributen (nach dem sie auf die rechte und linke Seite aufgeteilt wurden), wodurch aneinander grenzende "gleiche" Segmente verschmelzen (siehe Zeile 1879ff.) – funktioniert gut.

Es gibt im Pythonscript allerdings noch einen Edge Case, der momentan nicht abgefangen wird: Die verschmelzenden/benachbarten Segmente sollten in die gleiche Richtung zeigen, sonst könnten theoretisch Parkstreifen gemerged werden, die gar nicht zusammengehören, wodurch ein fehlerhafter "Tausch" der Eigenschaften an einem der Parkstreifen entsteht. Irgendwo im Schillerkiez gibt es eine Stelle, wo das passiert (zwei an der gleichen Kreuzung startende, aber in verschiedene Richtungen zeigende Linien, die jeweils auf der linken und rechten Seite die gleichen Parkattribute haben – was sich durch die verdrehte Richtung in der Realität aber auf verschiedene Straßenseiten bezieht). Kann aber auch sein, dass das mit der SQL-Methode kein Problem wäre, weil die Straßenseite als Attribut bekannt ist.

@tordans tordans changed the title Merge segments before calculation Processing: Merge segments before calculation Mar 31, 2023
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

3 participants