search

marhei / TrainCompany-Data

Datensammlung für TrainCompany
https://TrainCompany.de
16 stars 23 forks source link

Route Chomutov - Karlovy Vary - Cheb - Schirnding #659

Closed embix closed 2 years ago

embix commented 2 years ago

follow up jobs, once required routes are available (deferred to #684 )

embix commented 2 years ago

@C1710 import_brouter.py mal mit -v ausprobiert (aktuelles dev image, image id 40d5eb142f85)

marhei commented 2 years ago

Meinst du, dass du auch noch die Verbindung zwischen Schirnding und Cheb in den PR mit reinbekommst? 👀

embix commented 2 years ago

Meinst du, dass du auch noch die Verbindung zwischen Schirnding und Cheb in den PR mit reinbekommst?

Ja, dann wird das aber vermutlich nicht mehr heute fertig.

C1710 commented 2 years ago

@C1710 import_brouter.py mal mit -v ausprobiert (aktuelles dev image, image id 40d5eb142f85)

  • bei Chodov scheint er komische Koordinaten gezogen zu haben
  • bei Hlavno, Dasnice, Kynšperk nad Ohří, Nebanice und Tršnice stimmt der ermittelte CZ-Station Code nicht mit dem aus https://www.cd.cz/stanice überein (bei Hlavno, Dasnice, Kynšperk nad Ohří und Nebanice hatte er das auch als INFO geloggt - bei Tršnice allerdings nicht)
  • Constraint: Kurvigkeit darf 0.5 nicht überschreiten, weniger als 0.1 ist eigentlich schnurgerade Strecke (man könnte mit min-Radien und/oder Häufigkeit minimaler Radien arbeiten - bisher mache ich das nach Bauchgefühl - so ein grober Anhalt/Maßstab/Richtwert für Radius-Kurvigkeit wäre nicht verkehrt)
  • Anzahl und Länge der Bahnsteige ist noch Handarbeit 😄 (ja, ich weiß daß es inzwischen defaults gibt)
  1. Kann sein, dass es da wieder Stationen verwechselt hat. Ich ändere es aber vielleicht mal, dass es die Standorte mit denen aus Brouter überschreibt.
  2. Wenn es da was geloggt hat, nimmt es den Code von OpenStreetmaps, weil in den Datensätzen nichts entsprechendes gefunden wurde. Bei Tršnice kann es vielleicht sein, dass es ein etwas anderer Bahnhof ist
  3. Das läuft basierend auf dem Verhältnis Streckenlänge / Luftlinie. Danach ist 0.1 halt ziemlich gerade und erst 0.0 (bzw. 0.01) schnurgerade. Wenn da richtig Schlangen drin sind o.Ä., dann kann es auch schon mal über 0.4 kommen. Ich ändere das aber vielleicht demnächst, dass es eine vereinfachte Form als Grundlage nimmt.
  4. Bahnsteiginfos gibt es nur für DE, FR, selten auch UK und ganz selten aus OSM auch woanders
embix commented 2 years ago

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst? Bei größeren Stationsabständen und Gebirge könnte das dann einen größeren Unterschied machen.

C1710 commented 2 years ago

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst? Bei größeren Stationsabständen und Gebirge könnte das aber einen großen Unterschied machen.

Also es wird die Summe der Luftlinien zwischen den einzelnen Punkten der Route/des GPX Track-Segments verwendet. Zumindest immer, als ich das nachgesehen habe, hat es gepasst zu den Angaben auf brouter.de...

Die Höhe der Punkte wird nicht berücksichtigt.

C1710 commented 2 years ago

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst? Bei größeren Stationsabständen und Gebirge könnte das aber einen großen Unterschied machen.

Also es wird die Summe der Luftlinien zwischen den einzelnen Punkten der Route verwendet. Zumindest immer, als ich das nachgesehen habe, hat es immer gepasst zu den Angaben auf brouter.de...

Sobald es einen twistingFactor von > 0.01 ausgibt, muss die Strecke länger sein als die Luftlinie zwischen den zwei Halten. Also eigentlich kann das kein Problem sein.

embix commented 2 years ago

Die einzelnen Punkte der Route sind bei mir ja die Zwischenstops/Stationen - ich kann jetzt nicht mehr in die gpx Files gucken (schon gelöscht), ob er auch zwischen meinen Wegpunkten noch Hilfspunkte mitnimmt. In der Oberfläche hat er aber zumindest den Pfad 1:1 zur Bahnstrecke mitgeschwungen.

Ggf. mach ich am WE mal ein Testcase mit einer besonders geschwungenen Strecke - Brockenbahn oder sowas.

Aber zumindest kann ich aktuell nicht bestätigen, daß er deutlich daneben liegt - +/-1 (0.5) km ist bei int als Zielformat im json sowieso Rundungstoleranz.

C1710 commented 2 years ago

Die einzelnen Punkte der Route sind bei mir ja die Zwischenstops/Stationen - ich kann jetzt nicht mehr in die gpx Files gucken (schon gelöscht), ob er auch zwischen meinen Wegpunkten noch Hilfspunkte mitnimmt. In der Oberfläche hat er aber zumindest den Pfad 1:1 zur Bahnstrecke mitgeschwungen.

Ggf. mach ich am WE mal ein Testcase mit einer besonders geschwungenen Strecke - Brockenbahn oder sowas.

Aber zumindest kann ich aktuell nicht bestätigen, daß er deutlich daneben liegt - +/-1 (0.5) km ist bei int als Zielformat im json sowieso Rundungstoleranz.

Wenn es passt, dann sollte es ja korrekt sein :D

Vielleicht war eine Version bei ner älteren PR auch einfach ein bisschen kaputt oder so?

Also wenn du das exportierst, hast du i.d.R. ein paar Tausend Trackpoints (auf denen halt auch die angezeigte Route basiert).

embix commented 2 years ago

Also wenn du das exportierst, hast du i.d.R. ein paar Tausend Trackpoints (auf denen halt auch die angezeigte Route basiert).

Ja, dann sollte das passen.